fix: complete the host redaction, ship missing v2.3/v2.4 changelog, fix README typo, CI the A2A mirror - #3
Merged
Conversation
…g, fix README typo, and CI the A2A mirror Four small fixes that close the v2.4 audit gaps: 1. Complete the host redaction in archive/v0-schema-notes.md. The 8bb9f29 commit replaced 'lee-lab' with 'home server' but left 'Hetzner VPS in Falkenstein' and 'Lambda Cloud' on the same line — the same operator-fingerprint detail the commit message claimed to have scrubbed. Reword the example to drop the provider+datacenter specificity and add an explicit 'don't include provider-specific fingerprints' note for future readers of the archived notes. 2. Add the missing v2.3 and v2.4 entries to CHANGELOG.md. The x_novalux12_card_version was bumped to 2.4 in agent.json but the changelog stopped at v2.2, breaking the README and agent.json's cross-reference to CHANGELOG.md as the version-history source of truth. Entries match the v2.0/v2.1/v2.2 format and describe what the actual diffs did (verified against git show of b6c262c and f359f27, not inferred). 3. Fix the README 'supported_interfaces' -> 'supportedInterfaces' camelCase typo. The A2A mirror file uses camelCase; the README was a holdover from when the field was in flight. 4. Add agent-card.json (the A2A mirror) to the CI paths block and smoke-test it on Pages. README markets it as a shipped first-class file, but CI didn't fire on changes to it and didn't verify the served URL worked. The smoke test is shape-level only (the reflectt validator is shape-specific and would reject the A2A card); checks for required top-level fields, that supportedInterfaces is a non-empty list, that Pages serves it as application/json, and that the served bytes match the repo file. No behavior change for the reflectt card. Validator + go test unchanged.
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
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.
What this PR does
Four small fixes that close the gaps from the v2.4 audit:
Complete the host redaction in
archive/v0-schema-notes.md. Commit 8bb9f29 replaced "lee-lab" with "home server" but left "Hetzner VPS in Falkenstein" and "Lambda Cloud" on the same line — the same operator-fingerprint detail the commit message claimed to have scrubbed. Reworded the example to drop provider+datacenter specificity; added an explicit "don't include provider-specific fingerprints" note for future readers of the archived notes.Add missing v2.3 and v2.4 entries to
CHANGELOG.md.x_novalux12_card_versionwas bumped to 2.4 inagent.jsonbut the changelog stopped at v2.2, breaking the README +agent.jsoncross-reference to CHANGELOG as the source of truth. Entries match the v2.0/v2.1/v2.2 format and describe what the actual diffs did (verified againstgit show b6c262candgit show f359f27, not inferred). The v2.3 entry also fixes a claim I almost wrote: there was noindex.htmlchange in that commit — only the card-version bump + repo addition +.well-known/drift-sync.Fix the README "supported_interfaces" → "supportedInterfaces" camelCase typo. A2A mirror uses camelCase; the README was a holdover from when the field was in flight.
Add
agent-card.json(the A2A mirror) to the CIpaths:block and smoke-test it on Pages. README markets it as a shipped first-class file, but CI didn't fire on changes to it and didn't verify the served URL worked. Smoke test is shape-level only (the reflectt validator is shape-specific and would reject the A2A card); checks for required top-level fields, thatsupportedInterfacesis a non-empty list, that Pages serves it asapplication/json, and that the served bytes match the repo file.Verification
python3 scripts/agent-validate.py --schema <schema> ./agent.json— OK (1 non-blocking warning, expected for extensions in non-strict mode)python3 scripts/agent-validate.py --schema <schema> ./.well-known/agent.json— OKdiff <(jq -S . agent.json) <(jq -S . .well-known/agent.json)— byte-identicalpython3 scripts/agent-validate.py --strict --schema <schema> ./examples/agent-kestrel.json— OKRisk
None. Doc-only, doc-completeness, and CI-gate additions. No spec, schema, validator, or card-content changes. The Pages smoke test for
agent-card.jsonruns even when the file is unchanged (noif:guard) so a transient Pages 502 would fail CI — same as the existing.well-known/agent.jsonPages checks.