[kixelated]

Adds the three rules moq-dev/rust2#553 dropped, finishing quest/m0/context-rules.md:

• Required: the root-cause rule now covers flaky tests and names "looser assertion" as a forbidden workaround, instead of a separate line.
• Guidelines: "Match the code's existing conventions, patterns, and naming."
• Public API: "Name by role" gains short names under a module namespace, with an http::Client example.

assets/CONTEXT.md is 9,692 of the 10,000 character CONTEXT_LIMIT on main; moq-dev/quest#582 shortens it further. No room had to be made.

Choices made without asking: folding the flaky-test rule into the root-cause line to save characters, and the namespace example.

Public API / wire impact: none; hook context text only.

just check and just test pass.

Models

Size: S
Implementer: Claude, Opus 5.5, low effort

🤖 Generated with Claude Code

(Written by Claude Opus 5.5 with low effort)

Review

Rewritten by quest review after each round.

• Reviewer: Codex, gpt-6.1-sol, medium effort
• Session: 01a1c168-7f9d-73a2-9b64-bac448f3dfb1, round 1
• Head: 721d332
• Base: origin/main at 37963bf
• Score: 10/10, The three requested rules are incorporated, the context remains below its 10,000-character limit, and the completed quest has no remaining references; no actionable defects were found.
• Findings: none

[pullfrog]

  ^{｜ Review this ➔}

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 33 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: f74acf23-f6d7-45f0-b72b-e0b48fde1a75

Commits

Reviewing files that changed from the base of the PR and between 37963bf and 721d332.

Files selected for processing: 2

• assets/CONTEXT.md
• quest/m0/context-rules.md

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

[kixelated]

Automated review by review (OpenAI)

Reviewed head: 721d332

No actionable findings in the reviewed changes.

Direction: the three requested rules are incorporated into the relevant existing sections without duplicating the root-cause or naming rules. The complete CONTEXT.md fetched at this head is 9,692 characters, below the 10,000-character limit defined in hooks.ts, so this change does not depend on #582 landing to fit.

Verification limits: GitHub-only static review of both changed files, the surrounding context, and hook limit/consumption. No repository code, tests, builds, or live hook execution performed. The visible check/queue/lifecycle jobs are skipped and CodeRabbit is neutral; the PR author's reported local checks were not independently rerun.

(Written by review (OpenAI))