Skip to content

feat(sync): add opt-in per-target dropped summary#461

Merged
Chemaclass merged 2 commits into
mainfrom
feat/441-dropped-summary
Jun 18, 2026
Merged

feat(sync): add opt-in per-target dropped summary#461
Chemaclass merged 2 commits into
mainfrom
feat/441-dropped-summary

Conversation

@Chemaclass

Copy link
Copy Markdown
Owner

Summary

  • New sync.dropped-summary config (default off): prints a per-target summary of what each target could not fully emit — kinds with no native surface (dropped/unsupported) and kinds reachable only behind an opt-in key or source-dir only (downgraded).
  • Regroups the existing kind-grouped capability warnings and coverage notes by target, reading the same buffers read-only (no double-capture).
  • Honors the unchanged-since-last-sync digest suppression, so it stays quiet when nothing changed (consistent with the kind-grouped flushes) instead of re-printing every run.

Test plan

  • go test ./... (1255 pass)
  • agnostic-ai sync --check (clean), schemagen fresh, gofmt clean
  • Adversarial review workflow; fixed the confirmed major (digest-suppression inconsistency) + added CLI wiring test, dedup/sort/source-dir-branch coverage, and stdlib/wording cleanups.

Closes #441

sync.dropped-summary prints, per target, what the run could not fully
emit: kinds with no native surface (dropped) and kinds reachable only
behind an opt-in key or source-dir only (downgraded). It regroups the
existing kind-grouped capability warnings and coverage notes by target,
reading the same buffers without clearing them.

Off by default. Honors the unchanged-since-last-sync suppression so it
stays quiet when nothing changed, matching the kind-grouped flushes.

Related to #441
@Chemaclass Chemaclass added the enhancement New feature or request label Jun 18, 2026
@Chemaclass Chemaclass self-assigned this Jun 18, 2026
Hoist the negated condition into a dropsChanged bool instead of
!(a && b), per staticcheck De Morgan suggestion.

Related to #441
@Chemaclass Chemaclass merged commit aef4cb1 into main Jun 18, 2026
9 checks passed
@Chemaclass Chemaclass deleted the feat/441-dropped-summary branch June 18, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sync: per-kind/per-target 'what was dropped' summary for unsupported emissions

1 participant