Skip to content

feat: add skill sync drift summary#282

Merged
rogerchappel merged 2 commits intomainfrom
codex/skill-sync-drift-summary
May 1, 2026
Merged

feat: add skill sync drift summary#282
rogerchappel merged 2 commits intomainfrom
codex/skill-sync-drift-summary

Conversation

@rogerchappel
Copy link
Copy Markdown
Owner

Summary

  • add a pure helper for classifying skill sync drift
  • add driftStatus and requiresReview to skill sync status items
  • add aggregate needsReview, changed, and unknown counts for governance visibility

Verification

  • git diff --check
  • pnpm test src/lib/skill-sync-drift.test.ts
  • pnpm exec eslint src/lib/skill-sync-drift.ts src/lib/skill-sync-drift.test.ts src/app/api/skills/sync-status/route.ts
  • pnpm exec tsc --noEmit --pretty false
  • push hook: pnpm typecheck && pnpm build

Risk

Low. This is additive read-only response data and does not sync skills, mutate runtime config, touch secrets, or change existing status semantics.

@rogerchappel rogerchappel merged commit 2d80856 into main May 1, 2026
1 check passed
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