Skip to content

ci: add the Corbits Triage policy - #1

Closed
TheGreatAxios wants to merge 1 commit into
mainfrom
ci/triage-policy
Closed

TheGreatAxios wants to merge 1 commit into
mainfrom
ci/triage-policy

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11843 (https://linear.app/abklabs/issue/CL-11843/add-the-corbits-triage-policy-to-corbits-builder).

Changes

Adds .corbits-triage.yml:

  • Process rules:
    • A linked GitHub or Linear issue is required; a pull request without one is closed with a note to open an issue first. Other failures request changes.
    • Also checked: one change per pull request, Conventional Commits, CI green, no merge conflicts, no AI attribution, no hidden text, no duplicates.
    • Rejected outright: node_modules, .env.hub, .env.web and tmp/.
  • Sensitive paths: vendor/, patches/, migrations and auth are high; CI, deploy and bun.lock are medium.
  • Code rules on added code:
    • Fail: the convention preset (IIFEs, promise chains, empty catches, nested ternaries, ?? 0, ?? [], suppression comments), a catch without a binding, and imports of vendored Interchange source by path.
    • Ask a maintainer: hand-rolled setTimeout sleeps, the interface opening its own EventSource or WebSocket, and invented || [], ?? "", || "" and || 0 fallbacks.
  • Actions: approve passing pull requests from thegreataxios and brianjfox; assign unsure ones to thegreataxios.
  • Off: code review and size checks.

Verification

  • Triage's policy reader (policy-check-action) reports the file as valid.
  • Triage's local check (cli.ts check) on a throwaway commit failed all four code rules: conventions, errors-kept, interchange-owns-it and invented-fallbacks.
  • bun run check's pre-commit gates (format, lint, typecheck) passed.

@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11843

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Replaced by #2, on the Linear issue's branch.

@TheGreatAxios
TheGreatAxios deleted the ci/triage-policy branch October 31, 2026 07:54
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