Skip to content

docs(rules): gate output is not piped, no formatter where none is configured, README image version tags, bug-report dates - #544

Merged
bompus merged 3 commits into
mainfrom
fix/gate-output-piping
Nov 4, 2026
Merged

bompus merged 3 commits into
mainfrom
fix/gate-output-piping

Conversation

@bompus

@bompus bompus commented Nov 4, 2026

Copy link
Copy Markdown
Owner

Four one-sentence additions to the portable rules, each from a repeated problem in the owner's other repositories:

  • Gate output is not piped. Repository work (core text and its reference): run each check or test gate bare, or redirected to a log read after it exits, and never pipe it into head, tail or grep, because the consumer can hide a failure or end the gate early. This is the only addition to the always-on text.
  • No formatter where none is configured. repository-work.md only: a formatter reflows every file and buries the change; match the surrounding style by hand.
  • README image version tags. web-image-formats.md only: give each README image URL a ?v=N tag and bump N in the same commit that changes the asset's bytes. The claim behind it was checked: raw.githubusercontent.com answers with cache-control: max-age=300, and GitHub documents Camo as the caching proxy for images from other hosts.
  • Anchor a bug report's date. hr-diagnosing-bugs/references/reporting.md only: before agreeing a reported problem is still unfixed, anchor the report's date to released, merged-but-unreleased or in-progress state.

test/mixed-sections.test.mjs lists the formatter sentence as reference-only, which that test requires of every sentence that is not in the written section. A CHANGELOG bullet covers all four.

Checks: bun run check and GIT_CONFIG_GLOBAL=/dev/null GIT_CONFIG_SYSTEM=/dev/null bun test test skills (974 pass, 19 skip, 0 fail). An independent read (DeepSeek V4.1 Flash, max effort) found each sentence faithful to its source and no leak of private names or paths; its two wording nits are applied.

…o formatter where none is configured, README image version tags, anchoring a bug report's date to release state

Run check and test gates bare or redirected to a log (core Repository work and its reference); do not run a formatter where the repository configures none (repository-work reference); README image URLs carry a ?v=N tag bumped with the asset (web-image-formats reference); a report's date is anchored to released, merged-but-unreleased and in-progress state before agreeing a problem is unfixed (hr-diagnosing-bugs reporting reference).
@bompus bompus added author:anthropic Pull request written by a anthropic model author-model:sonnet-5-5 A model that wrote part of this pull request labels Nov 4, 2026

@bompus-ai-review bompus-ai-review 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.

Bompus AI Review (GPT-6.1 Terra, max reasoning; SWE-2 Max, max reasoning): 2 finding(s).

Comment thread rules/references/web-image-formats.md Resolved

@bompus-ai-review bompus-ai-review 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.

Bompus AI Review (GPT-6.1 Terra, max reasoning; SWE-2 Max, max reasoning): 3 finding(s).

Comment thread rules/references/web-image-formats.md Resolved
Comment thread rules/references/web-image-formats.md Resolved

@bompus-ai-review bompus-ai-review 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.

Bompus AI Review (GPT-6.1 Terra, max reasoning; SWE-2 Max, max reasoning): 0 finding(s).

Review 3: only high findings and above are posted from here on.

@bompus bompus closed this Nov 4, 2026
@bompus
bompus merged commit 4a134e0 into main Nov 4, 2026
2 checks passed
@bompus
bompus deleted the fix/gate-output-piping branch November 4, 2026 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:anthropic Pull request written by a anthropic model author-model:sonnet-5-5 A model that wrote part of this pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant