Skip to content

Clarify protocol guidance and refresh IETF comparison - #252

Merged
RobLe3 merged 9 commits into
mainfrom
docs/spec-guidance-ietf-20260925
Sep 25, 2026
Merged

RobLe3 merged 9 commits into
mainfrom
docs/spec-guidance-ietf-20260925

Conversation

@RobLe3

@RobLe3 RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Scope

  • Correct current onboarding, release, selection, authorization, transport, and maturity guidance without changing wire semantics.
  • Add a dated, official-source IETF comparison and REQ-1–17 mapping; retain older snapshots as historical.
  • Extend the existing offline comparison validator and profile fixture gate.

Two commits separate editorial guidance from standards/evidence changes. Published v1.10.17 tags and assets are unchanged.

Local validation

  • bash tools/run_profile_fixture_contract.sh (isolated local venv): pass.
  • python3 tools/check_spec_release_integrity.py: pass, 268 pinned files.
  • python3 tools/check_public_artifact_closure.py --all-public: pass.
  • python3 tools/check_public_prose.py --strict README.md GOVERNANCE.md CONTRIBUTING.md: 0 errors, one README boldface advisory.
  • Pre-1 strict freeze remains OPEN with six component reviews.

No release, deployment, production mutation, or independent qualification is asserted.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-25T11:06:11.390007Z 3b536d6 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 31af6039df

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/check_protocol_comparison.py Outdated
Comment thread standards/PROTOCOL_COMPARISON_2026-09-25.md
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 78aa557da7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/build_selection_review_bundle.py
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 550a306c55

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread standards/IICP_PROTOCOL_POSITIONING.md
Comment thread docs/architecture/decision-documentation-map.md
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 85374a46c0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/build_selection_review_bundle.py
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 46be3a250b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/build_standards_review_bundle.py
Comment thread spec/v1.9/release-integrity-manifest.json
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 27d7d78eb6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/build_standards_review_bundle.py
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7508a4d3a9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread VERSIONING.md
Comment thread standards/protocol-comparison-v1.json
@RobLe3

RobLe3 commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@RobLe3
RobLe3 merged commit e9c3002 into main Sep 25, 2026
1 check passed
@RobLe3
RobLe3 deleted the docs/spec-guidance-ietf-20260925 branch September 25, 2026 11:03

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3b536d63c4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

{
"id": "dawn",
"document": "charter-ietf-dawn-00-07",
"revision_date": "2026-09-11",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Remove the unverified DAWN revision date

The machine-readable inventory assigns the DAWN charter a revision_date of 2026-09-11 even though the same row explicitly says that the charter revision date was not separately established and cites only the group-status page. Consumers will treat this as verified revision metadata, and the validator merely checks its chronology. Cite an authoritative source that establishes this date or represent the revision date as unknown and update the validator accordingly.

Useful? React with 👍 / 👎.

# Local references needed to read the reviewed documents in isolation.
"CONTRIBUTING.md",
"GOVERNANCE.md",
"SPEC_RELEASE_PROCESS.md",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Include tools referenced by the newly bundled release guide

When either review archive is consumed standalone, the newly included SPEC_RELEASE_PROCESS.md directs reviewers to spec/v1.9/release-integrity-manifest.json, tools/check_spec_release_integrity.py, and tools/check_public_artifact_closure.py, but neither bundle inventory includes them. Fresh evidence beyond the earlier version-authority thread is that packaging the release guide introduced these new unresolved archive-relative references, which the link test misses because they are backtick paths rather than Markdown links; package them in both bundles and assert their presence.

Useful? React with 👍 / 👎.

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.

1 participant