Skip to content

Close the pre-commit docs gate and refresh catalog review output - #28

Merged
csark0812 merged 3 commits into
mainfrom
feat/catalog-refresh-compact-output
Sep 13, 2026
Merged

csark0812 merged 3 commits into
mainfrom
feat/catalog-refresh-compact-output

Conversation

@csark0812

Copy link
Copy Markdown
Owner

Summary

  • Refresh the local catalog on audit and validate, and print one review error per document with the files that invalidated it.
  • Close pre-commit holes: uncovered coverage-candidate paths fail on local and CI runs, --staged reads index bytes and requires attested papers to be staged, skill and policy changes run their audits in the hook, and skeleton init writes the hook with hash mode on.
  • Add a reviewCoverage map so files under configured globs must have an owning paper.

Test plan

  • bun test
  • bun run typecheck
  • bun run build
  • bun src/cli.ts audit self
  • Stage a code file with no review-deps owner and confirm validate changed --staged fails uncovered-changed-path
  • Attest a paper in the worktree only and confirm --staged fails stage-required

Made with Cursor

csark0812 and others added 3 commits September 13, 2026 14:08
Local audit and validate write the gitignored catalog instead of warning.
Review failures print one diagnostic per document with the files that triggered it.

Co-authored-by: Cursor <cursoragent@cursor.com>
…iew.

Co-authored-by: Cursor <cursoragent@cursor.com>
Production ESM must stay a coverage candidate. Plugin output stays excluded under .skeleton/plugins.

Co-authored-by: Cursor <cursoragent@cursor.com>
@csark0812
csark0812 merged commit ee85222 into main Sep 13, 2026
2 checks passed
@csark0812
csark0812 deleted the feat/catalog-refresh-compact-output branch September 13, 2026 21:37
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.

1 participant