Skip to content

fix(package-index): align validation with package manager config - #149

Open
gbrogers wants to merge 2 commits into
mainfrom
gbrogers/runneth-package-index
Open

fix(package-index): align validation with package manager config#149
gbrogers wants to merge 2 commits into
mainfrom
gbrogers/runneth-package-index

Conversation

@gbrogers

@gbrogers gbrogers commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Updates the Runneth package-index validator to match the current package-manager contract: package-root package.json manifests, installPolicy, backend-github package sources, and non-semver version strings.
  • Updates managed-sync fleet approval detection to account for installPolicy and updatePolicy changes.
  • Refreshes the validation workflow comment so it describes the current catalog, package-index, and site jobs.

Validation

  • node --test scripts/validate-runneth-package-index.mjs
  • Temporary non-empty package-index fixture with backend-github source, package.json manifest, installPolicy, package_instruction, and non-semver version string

Notes

  • node --test scripts/validate-catalog.mjs still fails on existing catalog data: runneth-classic is built but missing from catalog/excluded, and add-roles-permissions/install-config.json is missing changelog data.

@gbrogers
gbrogers requested a review from a team July 2, 2026 15:55
@gbrogers
gbrogers force-pushed the gbrogers/runneth-package-index branch from 7a7ab01 to b156873 Compare July 2, 2026 15:59
@gbrogers
gbrogers requested a review from ioanahayman July 2, 2026 16:00
@gbrogers
gbrogers requested a review from thedumsieffect July 2, 2026 16:00

@thedumsieffect thedumsieffect left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what a great piece of code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants