Skip to content

docs: cite launcher-standard_praxis.deed, not the deleted a2ml - #228

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/ac1-cite-launcher-standard-praxis-deed
Sep 22, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
docs/ac1-cite-launcher-standard-praxis-deed

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

What

This repo's launcher descriptor cited launcher-standard.a2ml, which
standards#952 deleted.

The change

Comment lines only:

from to
# Compliant with launcher-standard.a2ml v0.3.0 # Compliant with launcher-standard_praxis.deed v0.4.0
# Per launcher-standard.a2ml [version-output] section # Per launcher-standard_praxis.deed (version-output ...) clause
# Per launcher-standard.a2ml [runtime] section: # Per launcher-standard_praxis.deed (runtime ...) clause:

The old citation was wrong twice over: wrong file, and [section] bracket
notation the DEED grammar does not have. DEED is s-expression based — the real
clause heads are (version-output ...) and (runtime ...), verified in
hyperpolymath/standards launcher/launcher-standard_praxis.deed.

Why v0.4.0 is earned, not asserted

This descriptor does not resolve the standard — (resolution (standard-search ...))
does not apply to it. So naming the current version is a statement of
conformance, not a claim about a resolution path.

Scope

Comment-only. No behaviour change, and independent of the file extension —
migrating .launcher.a2ml → .deed is a separate workstream gated behind
hyperpolymath/launch-scaffolder#40 under the dual-accept rule, and is
deliberately not attempted here.

Verification

git grep -c 'launcher-standard\.a2ml' over this repo returns 0.

Refs: hyperpolymath/standards#960 (AC1), standards#837, owner ruling D73-C.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WPSJ7fBhVAMcpSffCBWUDo

Summary by CodeRabbit

  • Documentation
    • Updated launcher configuration references to the v0.4.0 launcher specification.
    • Revised compliance, PID/log-file requirement, and version-output comments.
    • Runtime settings remain unchanged.

standards#952 deleted launcher/launcher-standard.a2ml. This descriptor
still named it, and used [section] bracket notation the DEED grammar does
not have. Cites the live launcher-standard_praxis.deed at its current
version and uses DEED clause notation.

v0.4.0 is a statement of conformance, not a resolution path: this
descriptor does not resolve the standard, so it names the current version
rather than a version it loads.

Comment-only; no behaviour change, and independent of the file extension
(the .deed extension migration is gated behind launch-scaffolder#40).

Refs: hyperpolymath/standards#960 (AC1), standards#837, D73-C.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WPSJ7fBhVAMcpSffCBWUDo
@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 13614ef0-700c-406b-9be1-3c3f8a844edc

📥 Commits

Reviewing files that changed from the base of the PR and between 73c1305 and 8952cb3.

📒 Files selected for processing (1)
  • burble.launcher.a2ml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The launcher configuration comments now reference launcher-standard_praxis.deed v0.4.0 and its clauses. The PID file, log file, and wait-for-URL timeout values remain unchanged.

Changes

Launcher specification references

Layer / File(s) Summary
Update launcher specification comments
burble.launcher.a2ml
The compliance, runtime requirement, and version-output comments now reference launcher-standard_praxis.deed v0.4.0. The configuration directives and values remain unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 8952c

This PR updates launcher specification comments without changing runtime behavior or configuration values, so it is merge-ready.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the change, reason, scope, and verification. It does not include the repository-required RSR Quality Checklist or explicitly use the Testing section. Add the RSR Quality Checklist and mark each applicable item. Add a Testing section that records the verification command and result. Add a Screenshots section or state that screenshots are not applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: updating the documentation citation to the replacement DEED standard.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡

❌ Error running CI fixer.

  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Rate Limit Exceeded

@hyperpolymath have exceeded the limit for the number of chat messages per hour. Please wait 14 minutes and 13 seconds before sending another message.

@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Rate Limit Exceeded

@hyperpolymath have exceeded the limit for the number of chat messages per hour. Please wait 13 minutes and 44 seconds before sending another message.

@hyperpolymath
hyperpolymath merged commit e36ecdd into main Sep 22, 2026
45 of 48 checks passed
@hyperpolymath
hyperpolymath deleted the docs/ac1-cite-launcher-standard-praxis-deed branch September 22, 2026 14:40
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