Skip to content

Fix title and subtitle body duplication - #542

Merged
edithatogo merged 2 commits into
masterfrom
codex/t18-01-title-subtitle-dedup
Aug 10, 2026
Merged

edithatogo merged 2 commits into
masterfrom
codex/t18-01-title-subtitle-dedup

Conversation

@edithatogo

Copy link
Copy Markdown
Owner

Summary

  • remove exact leading title and subtitle repetitions at the shared preparation boundary
  • preserve non-matching headings, standfirsts, later matches, and heading-derived title fallback
  • correct examples and document the canonical user/agent Markdown shape
  • register and archive P18/T18-01 with native issues and Project evidence

Validation

  • npx vitest run src/publish/prepare.test.ts: 14 passed
  • npm run typecheck: passed
  • npm test: 117 files, 933 tests passed
  • npm run contracts:check: passed
  • npm run governance:github-programme: 13 phases, 47 tracks, 194 tasks, 268 Project items valid

Closes #537

Copilot AI lite review requested due to automatic review settings August 10, 2026 13:59
@cursor

cursor Bot commented Aug 10, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/publish/prepare.ts 94.44% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@edithatogo
edithatogo merged commit 59e5dfc into master Aug 10, 2026
32 of 34 checks passed
@edithatogo
edithatogo deleted the codex/t18-01-title-subtitle-dedup branch August 10, 2026 14:24
edithatogo added a commit that referenced this pull request Aug 10, 2026
## Summary
- address the actionable Codecov comment on #542
- pass the already-narrowed leading block into the removal helper
- remove the unreachable defensive branch instead of adding an
impossible test

## Validation
- focused prepare suite: 14 passed
- focused V8 coverage: no uncovered `prepare.ts` line reported
- Biome CI: passed
- TypeScript typecheck: passed

Follow-up to #542 and #537.

Co-authored-by: Dylan Mordaunt (ISLHD) <Dylan.Mordaunt@health.nsw.gov.au>
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.

Prevent title and subtitle duplication in published post bodies

3 participants