Skip to content

House standards: comments rule revised (#70 part 1); the file is style-only - #71

Merged
mcarter-astronautdev merged 1 commit into
mainfrom
comments-standard
Oct 1, 2026
Merged

mcarter-astronautdev merged 1 commit into
mainfrom
comments-standard

Conversation

@mcarter-astronautdev

Copy link
Copy Markdown
Member

Closes the first half of #70 and records the decision on Avara #226.

  • Kit: the Code comments bullet in claude-standards.md becomes the guidebooks-stay, provenance-goes rule Maria kept at Avara, written generically (client, app, Figma) and shorter than the proposal. The header says the file is style-only.
  • CLAUDE.md: invariant: nothing load-bearing goes in the standards file, because on a PR run the action restores CLAUDE.md from the base branch but the @ import resolves to the PR head.
  • Handoff: the open decision closes; part 2 of Revise the Code comments standard: wayfinding and headers stay, provenance goes #70 (the implementer prompt) rides with the next reusable change.

…s; the file is style-only

The comments bullet follows what Maria kept in the Avara #215 revert (#70 part 1), generic for the
fleet and shorter than the proposal. The style-only rule records the accepted PR-run gap: the
action restores CLAUDE.md from the base branch but the import resolves to the PR head.
@macroscopeapp

macroscopeapp Bot commented Oct 1, 2026

Copy link
Copy Markdown

Approvability

Verdict: Approved at da7f0fe

Macroscope's review found this PR approvable — This PR revises shared Claude comment/style guidance and records the associated documentation decision, without changing executable workflows, permissions, schemas, deployment, or customer-facing runtime behavior. Its fleet-wide effect is limited to future implementer guidance, making the change bounded and low risk.

You can add or adjust custom eligibility rules. Learn more.

@mcarter-astronautdev
mcarter-astronautdev merged commit 71e869c into main Oct 1, 2026
4 checks passed
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