minor requested changes

Author: savil

internal/devconfig/packages.go

@@ -248,7 +248,8 @@ func NewPackage(name string, values map[string]any) Package {
 // If the package has a list of platforms, it is enabled only on those platforms.
 // If the package has a list of excluded platforms, it is enabled on all platforms
 // except those.
-func (p *Package) enabledOnPlatform(platform string) bool {
+func (p *Package) IsEnabledOnPlatform() bool {
+	platform := nix.MustGetSystem()
 	if len(p.Platforms) > 0 {
 		for _, plt := range p.Platforms {
 			if plt == platform {
@@ -273,10 +274,6 @@ func (p *Package) VersionedName() string {
 	return name
 }
 
-func (p *Package) IsEnabledOnPlatform() bool {
-	return p.enabledOnPlatform(nix.MustGetSystem())
-}
-
 func (p *Package) UnmarshalJSON(data []byte) error {
 	// First, attempt to unmarshal as a version-only string
 	var version string

internal/impl/packages.go

@@ -243,11 +243,7 @@ func (d *Devbox) ensurePackagesAreInstalled(ctx context.Context, mode installMod
 	}
 
 	// Update lockfile with new packages that are not to be installed
-	cfgPkgs, err := d.ConfigPackages()
-	if err != nil {
-		return err
-	}
-	for _, pkg := range cfgPkgs {
+	for _, pkg := range d.configPackages() {
 		if err := pkg.EnsureUninstallableIsInLockfile(); err != nil {
 			return err
 		}