docs(release): polish CONNECTOR_RELEASE_CONTRACT.md round-2 review patches (Story 44.4 follow-up)#68
Merged
ALLiDoizCode merged 1 commit intomainfrom May 9, 2026
Conversation
…tches Six small content fixes deferred from Story 44.4's round-2 review cycle (connector#67 merged before these could be pushed): - P3: line 14 PR #62 → #63 (matches the multi-arch sentence two lines below and the connector#63/#64/#65 series in sprint-status) - P4: pin cosign opener to "v3.6.0 (cut after PR #66 merged)" instead of the ambiguous "first release after PR #66" wording - P5: ## Verification lede "Two mechanisms" → "Three mechanisms" + remove blank line so item 3 renders contiguously with items 1-2 - P6: add RFC-0027 reference for the ILP packet wire-format MAJOR-bump rule (Story 44.4 Dev Notes promised this cross-link but the published doc did not include it) - P7: ## Staying current gh CLI fenced block opens with ```bash to match every other code block in the doc - Q1: ## Verification item 3 anchors cwd ("from the toon-protocol/connector repo root, with toon-protocol/town cloned as ../town") and replaces the unrunnable /path/to/town/... placeholder with ../town/packages/sdk/... Sibling town mirror PR will land in lock-step to keep the body-equivalence diff clean. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Six small content fixes deferred from Story 44.4's round-2 review cycle (connector#67 merged before these could be pushed). All fixes were verified body-clean during the original review (local diff returned empty against the patched mirror); shipping them now bundled into one connector PR + one sibling town mirror PR.
Tracked in townhouse repo at
_bmad-output/implementation-artifacts/deferred-work.md(post-merge re-strategy section for Story 44.4) and surfaced in the Epic 44 retrospective at_bmad-output/implementation-artifacts/epic-44-retro-2026-05-08.md.Patches
## Artifactsline 14PR #62→PR #63(matches the multi-arch sentence two lines below + the connector#63/#64/#65 series in sprint-status)## Supply-chain signingopener## Verificationlede + items 1-3## References[Interledger Protocol V4 (RFC 0027)]for the ILP packet wire-format MAJOR-bump rule (Story 44.4 Dev Notes promised this cross-link; published doc did not include it)## Staying current## Verificationitem 3Sibling town mirror PR
A sibling PR will land in
toon-protocol/townto mirror this body-identical content intopackages/sdk/CONNECTOR_RELEASE_CONTRACT.md. Both PRs should land same-day to keep the body-equivalence diff clean:```bash
diff CONNECTOR_RELEASE_CONTRACT.md \
<(tail -n +4 ../town/packages/sdk/CONNECTOR_RELEASE_CONTRACT.md)
Expected: empty
```
Commit type
`docs(release): ...` — semantic-release maps this to no version bump, which is correct (these are doc-only polish fixes; the contract itself was already published in v3.6.0+ via PR #67).
Test plan
CONNECTOR_RELEASE_CONTRACT.md[Unreleased]entry added under### Documentation🤖 Generated with Claude Code