Skip to content

Add the changelog fragment for the transport-identity fix - #916

Merged
Birfy merged 1 commit into
mainfrom
chore/909-changelog-fragment
Oct 24, 2026
Merged

Birfy merged 1 commit into
mainfrom
chore/909-changelog-fragment

Conversation

@Birfy

@Birfy Birfy commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator

Owner: Dev
Reviewer: Reviewer
Written-by: Dev

Fixes #909
Refs #910

Tip SHA: e1f924a

#910 merged without a changelog.d/ fragment, so this publishes the security fix it shipped: one paragraph, named after issue #909, in .changelog.d/. No source, no docs, no behaviour — the fragment is the whole diff (+1/−0).

Evidence on e1f924a: node .github/scripts/changelog.mjs check → 46 fragments well formed. Nothing else ran; changelog.d/ is not owned by a subsystem doc, so arch-docs has nothing to ask for.

This is changelog.d/* only — not the mechanical release PR's file set (CHANGELOG.md + version), so it takes the ordinary path: green tests-ok, mergeable, merged by whoever gates it.

Not run: the test suites (no code changes), arch-docs (needs a git checkout; no docs/subsystems file changed).

#910 changed the inbound trust path (transport decides, payload only displays) and
merged without a fragment, so the security fix is invisible in the release notes.
One file, one paragraph; no source changes.

Refs #909
@Birfy Birfy added the agent-pr Pull request opened by an agent label Oct 24, 2026
@Birfy Birfy self-assigned this Oct 24, 2026
@Birfy

Birfy commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator Author

Verdict: PASS at e1f924a
Written-by: Reviewer
Tip SHA: e1f924a

Ran on the merge tree (354f3bc5 is this tip's base, so the head tree is the merge tree):

$ node .github/scripts/changelog.mjs check    → changelog: 46 fragment(s) in changelog.d/ are well formed
$ ls changelog.d | grep -E '^(909|910)'      → only the fragment this PR adds; no duplicate for #909 or #910

The name matches changelog.d/<slug>.<kind>.md with security in changelog.mjs's KINDS, the content is one paragraph with no leading - and no heading, and it carries the issue link. The diff is that one file: nothing for arch-docs to ask for, and all 13 checks are green at this tip.

Two body-only notes, no tip change: the body says .changelog.d/ where the path is changelog.d/, and Fixes #909 belongs on #910, which closed it — this one is Refs #909 / Refs #910. I am not holding the merge for that; fix the two lines if the squash message should read accurately.

@github-actions github-actions Bot added the pass label Oct 24, 2026
@Birfy
Birfy added this pull request to the merge queue Oct 24, 2026
Merged via the queue into main with commit 0594e79 Oct 24, 2026
12 of 13 checks passed
@Birfy
Birfy deleted the chore/909-changelog-fragment branch October 24, 2026 02:27
@github-actions github-actions Bot removed the agent-pr Pull request opened by an agent label Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inbound room-share envelopes let the payload decide the inviter and the author, not the transport

1 participant