Skip to content

Fix validation of shared source repositories - #32

Merged
steve-calvert-glean merged 2 commits into
mainfrom
codex/scope-validator-to-output
Aug 11, 2026
Merged

Fix validation of shared source repositories#32
steve-calvert-glean merged 2 commits into
mainfrom
codex/scope-validator-to-output

Conversation

@steve-calvert-glean

Copy link
Copy Markdown
Contributor

Summary

  • have each target validator report the marketplace files and plugin directories it owns
  • scan only those native output paths for surviving partial references
  • ignore authored shared/**, workflow examples, dependencies, and other unrelated files that coexist with root-level target output
  • preserve detection of stale or unmanaged partial tags inside emitted plugins

Why

Pluginpack 0.11 recommends repositories where authored shared/** sources and generated marketplaces coexist under one root. pluginpack validate --dir <root> currently walks the entire root, so valid authored partials and even Pluginpack’s own node_modules can fail validation.

Validation

  • regression test reproduces source/workflow/dependency false positives in the recommended single-repo layout
  • existing stale-output partial test remains green
  • direct Claude, Cursor, and Codex validation passes against glean-cookbook without its staging wrapper
  • mise exec -- npm run test:all (202 passed, 1 skipped)

@steve-calvert-gleansteve-calvert-glean added the bug Something isn't working label Aug 11, 2026
Comment threadsrc/targets/validation-shared.ts
@steve-calvert-glean
steve-calvert-glean merged commit 670cf97 into mainAug 11, 2026
3 checks passed
@steve-calvert-glean
steve-calvert-glean deleted the codex/scope-validator-to-output branch August 11, 2026 03:15
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@steve-calvert-glean