fix: compact idempotent summary rows, trailing newline, well-formed rows, counts/pluralization (closes #417) by 0xLeif · Pull Request #447 · CorvidLabs/spec-sync

0xLeif · 2026-07-24T16:52:43Z

Summary

Fixes #417 completely for compact and the related archive-tasks output path.

Makes changelog compaction idempotent with an explicit provenance marker and cumulative summary handling.
Preserves exact LF, CRLF, mixed-ending, trailing-newline, and no-final-newline bytes.
Parses Markdown tables without truncating escaped pipes or code-span pipes and emits the exact original table width.
Fails closed without writes for malformed, duplicate, overflowed, indented-separator, or reordered marked-summary state.
Reports truthful compacted/kept counts and singular/plural labels.
Honors text, JSON, Markdown, and GitHub formats for both commands with valid structured output and no ANSI leakage.
Uses plan/stage/publish/rollback transactions so planning or staging failures write nothing and late publish failures are reported truthfully.
Safely renders unusual paths, including literal Unix backslashes, backslash-before-pipe names, backticks, HTML characters, control characters, and bidi controls.

Lifecycle and review

SpecSync change: CHG-0065-make-issue-417-changelog-compaction-idempotent-and-provide-truthful-portable-str
Definition approved by the user at digest 4cb1f669e6892906b7ed9d78d872de1eb133de0725ceac4ef262ff7451d99984.
Two independent reviewer agents checked every issue-body acceptance row plus adversarial security, compatibility, and regression behavior.
No unresolved high- or medium-severity findings remain.
The old PR history was replaced with an eight-commit dependency-ordered lifecycle history based directly on current main.

Verification

fledge lanes run verify
- 2,002 unit tests passed
- 329 integration tests passed
- release build passed
specsync check --strict --require-coverage 100 --force
- 62/62 specs passed
- 105/105 files covered
- 110,054/110,054 LOC covered
Private CorvidLabs/spec-sync-sandbox disposable-clone replay
- first compact/archive apply succeeded
- second run was a byte-identical no-op
- JSON stayed parseable
- rendered Markdown preserved adversarial path names in one table cell
Signed Attest provenance recorded at 0.95 confidence with tests passed and human definition approval.
Augur returned review at risk 40 and did not return BLOCK.

Test plan

chatgpt-codex-connector · 2026-07-24T16:52:49Z

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

github-actions

❌ Corvin says...

      _
    <(;\  .oO(oh no...)
     |/(\
      \(\\
      " "\\

"I'm pecking through the errors..."

CI Summary

Check	Status
Validate action.yml	✅ Passed
Packaged Action Consumer	✅ Passed
Dependency Audit	✅ Passed
Code Coverage	✅ Passed
Format Check	❌ failure
Docs Site	✅ Passed
Spec Validation	❌ failure
Tests (build, test, clippy)	✅ Passed
VS Code Extension	✅ Passed

📋 Spec Validation Details

❌ SpecSync: Failed

Metric	Value
Specs checked	62
Passed	62
Errors	1
Warnings	0
File coverage	100% (105/105)
LOC coverage	100% (87065/87065)

Errors

.specsync/sdd.json

meaningful changed paths are not covered by an active change: src/commands/compact.rs, src/compact.rs

Action Items

Review and fix -- meaningful changed paths are not covered by an active change: src/commands/compact.rs, src/compact.rs

Generated by specsync · Run specsync check --format github to reproduce

_{Powered by corvid-pet}

github-actions

❌ Corvin says...

      _
    <(;\  .oO(oh no...)
     |/(\
      \(\\
      " "\\

"Caw... your imports are all over the place."

CI Summary

Check	Status
Validate action.yml	✅ Passed
Packaged Action Consumer	✅ Passed
Dependency Audit	✅ Passed
Code Coverage	✅ Passed
Format Check	✅ Passed
Docs Site	✅ Passed
Spec Validation	❌ failure
Tests (build, test, clippy)	❌ failure
VS Code Extension	✅ Passed

📋 Spec Validation Details

❌ SpecSync: Failed

Metric	Value
Specs checked	62
Passed	62
Errors	1
Warnings	0
File coverage	100% (105/105)
LOC coverage	100% (87284/87284)

Errors

.specsync/sdd.json

meaningful changed paths are not covered by an active change: specs/cli/cli.spec.md, specs/cli/context.md, specs/cli/requirements.md, specs/cli/tasks.md, specs/cli/testing.md, specs/cmd_archive_tasks/cmd_archive_tasks.spec.md, specs/cmd_archive_tasks/context.md, specs/cmd_archive_tasks/requirements.md, specs/cmd_archive_tasks/tasks.md, specs/cmd_archive_tasks/testing.md, specs/cmd_compact/cmd_compact.spec.md, specs/cmd_compact/context.md, specs/cmd_compact/requirements.md, specs/cmd_compact/tasks.md, specs/cmd_compact/testing.md, specs/compact/compact.spec.md, specs/compact/context.md, specs/compact/requirements.md, specs/compact/tasks.md, specs/compact/testing.md, src/commands/archive_tasks.rs, src/commands/compact.rs, src/compact.rs, src/main.rs, tests/integration/commands.rs

Action Items

Review and fix -- meaningful changed paths are not covered by an active change: specs/cli/cli.spec.md, specs/cli/context.md, specs/cli/requirements.md, specs/cli/tasks.md, specs/cli/testing.md, specs/cmd_archive_tasks/cmd_archive_tasks.spec.md, specs/cmd_archive_tasks/context.md, specs/cmd_archive_tasks/requirements.md, specs/cmd_archive_tasks/tasks.md, specs/cmd_archive_tasks/testing.md, specs/cmd_compact/cmd_compact.spec.md, specs/cmd_compact/context.md, specs/cmd_compact/requirements.md, specs/cmd_compact/tasks.md, specs/cmd_compact/testing.md, specs/compact/compact.spec.md, specs/compact/context.md, specs/compact/requirements.md, specs/compact/tasks.md, specs/compact/testing.md, src/commands/archive_tasks.rs, src/commands/compact.rs, src/compact.rs, src/main.rs, tests/integration/commands.rs

Generated by specsync · Run specsync check --format github to reproduce

_{Powered by corvid-pet}

github-actions

✅ Corvin says...

      _
    <(^\  .oO(Caw! ^v^)
     |/(\
      \(\\
      " "\\

"Caw! Your code sparkles like a dropped french fry."

CI Summary

Check	Status
Validate action.yml	✅ Passed
Packaged Action Consumer	✅ Passed
Dependency Audit	✅ Passed
Code Coverage	✅ Passed
Format Check	✅ Passed
Docs Site	✅ Passed
Spec Validation	✅ Passed
Tests (build, test, clippy)	✅ Passed
VS Code Extension	✅ Passed

📋 Spec Validation Details

✅ SpecSync: Passed

Metric	Value
Specs checked	62
Passed	62
Errors	0
Warnings	0
File coverage	100% (105/105)
LOC coverage	100% (110054/110054)

Generated by specsync · Run specsync check --format github to reproduce

_{Powered by corvid-pet}

github-actions

✅ Corvin says...

      _
    <(^\  .oO(Caw! ^v^)
     |/(\
      \(\\
      " "\\

"That's a nice looking export you've got there."

CI Summary

Check	Status
Validate action.yml	✅ Passed
Packaged Action Consumer	✅ Passed
Dependency Audit	✅ Passed
Code Coverage	✅ Passed
Format Check	✅ Passed
Docs Site	✅ Passed
Spec Validation	✅ Passed
Tests (build, test, clippy)	✅ Passed
VS Code Extension	✅ Passed

📋 Spec Validation Details

✅ SpecSync: Passed

Metric	Value
Specs checked	62
Passed	62
Errors	0
Warnings	0
File coverage	100% (105/105)
LOC coverage	100% (110054/110054)

Generated by specsync · Run specsync check --format github to reproduce

_{Powered by corvid-pet}

github-actions

✅ Corvin says...

      _
    <(^\  .oO(Caw! ^v^)
     |/(\
      \(\\
      " "\\

"Looking sharp! Like a beak should be."

CI Summary

Check	Status
Validate action.yml	✅ Passed
Packaged Action Consumer	✅ Passed
Dependency Audit	✅ Passed
Code Coverage	✅ Passed
Format Check	✅ Passed
Docs Site	✅ Passed
Spec Validation	✅ Passed
Tests (build, test, clippy)	✅ Passed
VS Code Extension	✅ Passed

📋 Spec Validation Details

✅ SpecSync: Passed

Metric	Value
Specs checked	62
Passed	62
Errors	0
Warnings	0
File coverage	100% (105/105)
LOC coverage	100% (110054/110054)

Generated by specsync · Run specsync check --format github to reproduce

_{Powered by corvid-pet}

0xLeif requested a review from a team as a code owner July 24, 2026 16:52

0xLeif requested review from 0xGaspar, Kyntrin and tofu-ux July 24, 2026 16:52

github-actions Bot requested changes Jul 24, 2026

View reviewed changes

github-actions Bot requested changes Jul 27, 2026

View reviewed changes

0xLeif added 8 commits July 26, 2026 23:27

docs(lifecycle): define CHG-0065 issue 417 maintenance correctness

20ca53a

docs(lifecycle): record CHG-0065 definition approval

ff8770e

docs(lifecycle): start CHG-0065 implementation

c3f6088

fix(compact): resolve issue 417 maintenance correctness

1075f0c

docs(specs): document maintenance report exports and evidence

872b57c

fix(compact): close adversarial issue 417 gaps

5b1760e

fix(compact): preserve backslash-pipe markdown paths

065fcd1

fix(compact): reject indented and reordered changelog state

d6c12fd

0xLeif force-pushed the fix/417-compact-idempotent branch from 025b315 to d6c12fd Compare July 27, 2026 06:53

github-actions Bot previously approved these changes Jul 27, 2026

View reviewed changes

0xLeif added 2 commits July 27, 2026 01:15

chore(lifecycle): record CHG-0065 verification evidence

c5c5bf3

chore(lifecycle): verify CHG-0065

204e548

0xLeif dismissed github-actions[bot]’s stale review via 204e548 July 27, 2026 07:32

github-actions Bot previously approved these changes Jul 27, 2026

View reviewed changes

0xLeif added 2 commits July 27, 2026 07:30

Fix: normalize CHG-0065 semantic deltas

fb1eff7

Update: accept CHG-0065

332d2e4

0xLeif dismissed github-actions[bot]’s stale review via 332d2e4 July 27, 2026 14:22

github-actions Bot approved these changes Jul 27, 2026

View reviewed changes

0xLeif merged commit c74b19e into main Jul 27, 2026
20 checks passed

0xLeif deleted the fix/417-compact-idempotent branch July 27, 2026 14:42

0xLeif mentioned this pull request Jul 27, 2026

chore(lifecycle): archive CHG-0065 after #447 #465

Merged

2 tasks

0xLeif added a commit that referenced this pull request Jul 27, 2026

chore(lifecycle): archive CHG-0065 after PR #447 merge (#465)

a55a744

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: compact idempotent summary rows, trailing newline, well-formed rows, counts/pluralization (closes #417)#447

fix: compact idempotent summary rows, trailing newline, well-formed rows, counts/pluralization (closes #417)#447
0xLeif merged 12 commits into
mainfrom
fix/417-compact-idempotent

0xLeif commented Jul 24, 2026 •

edited

Loading

Uh oh!

chatgpt-codex-connector Bot commented Jul 24, 2026

Uh oh!

github-actions Bot left a comment

Uh oh!

github-actions Bot left a comment

Uh oh!

github-actions Bot left a comment

Uh oh!

github-actions Bot left a comment

Uh oh!

github-actions Bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

0xLeif commented Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Lifecycle and review

Verification

Test plan

Uh oh!

chatgpt-codex-connector Bot commented Jul 24, 2026

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

❌ Corvin says...

CI Summary

❌ SpecSync: Failed

Errors

Action Items

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

❌ Corvin says...

CI Summary

❌ SpecSync: Failed

Errors

Action Items

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

✅ Corvin says...

CI Summary

✅ SpecSync: Passed

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

✅ Corvin says...

CI Summary

✅ SpecSync: Passed

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

✅ Corvin says...

CI Summary

✅ SpecSync: Passed

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

0xLeif commented Jul 24, 2026 •

edited

Loading