From b224490fb6dc4015c7ed07687e3e8fbc4e82e63a Mon Sep 17 00:00:00 2001 From: Luke Curley <432854+kixelated@users.noreply.github.com> Date: Fri, 6 Nov 2026 16:49:29 -0800 Subject: [PATCH] quest(m0): plan three generic rules for the shared context Co-Authored-By: Claude Opus 5.5 --- quest/m0/README.md | 1 + quest/m0/context-rules.md | 31 +++++++++++++++++++++++++++++++ 2 files changed, 32 insertions(+) create mode 100644 quest/m0/context-rules.md diff --git a/quest/m0/README.md b/quest/m0/README.md index 385ec2b4..c367516e 100644 --- a/quest/m0/README.md +++ b/quest/m0/README.md @@ -105,3 +105,4 @@ ownership-manifest design: - [Agents get the dev shell from user settings](/quest/m0/user-dev-shell/README.md) - maintainers' own Claude and Codex settings load the dev shell; repositories drop dev-shell hooks, Quest worktree hooks, and `nix develop` instructions, and `quest init` writes no hooks - [Session checkouts released](/quest/m0/session-checkouts.md) - a Claude session's own checkout is released when the session ends and reclaimed on resume - [claude rm upstream report](/quest/m0/claude-rm-upstream.md) - posting approved: ask Claude Code to run `WorktreeRemove` instead of advising an unlock +- [Three generic rules in the shared context](/quest/m0/context-rules.md) - `quest context` adds matching conventions, never loosening a flaky test, and short names under a namespace diff --git a/quest/m0/context-rules.md b/quest/m0/context-rules.md new file mode 100644 index 00000000..cf89b3c0 --- /dev/null +++ b/quest/m0/context-rules.md @@ -0,0 +1,31 @@ +# [S] Three generic rules in the shared context + +## Goal + +`quest context` gives every repository three engineering rules that moq-dev/rust2 used to +carry itself, each said once in the section it belongs to: + +- Match the code's existing conventions, patterns, and naming. +- Never pass a flaky test by loosening its assertion; fix the cause. +- Short names under a module namespace, rather than repeating the module in each name. + +## Plan + +[moq-dev/rust2#553](https://github.com/moq-dev/rust2/pull/553) replaced rust2's own copies +of the shared rules with `quest context`, dropping these because `assets/CONTEXT.md` had +no equivalent. Word each to fit its neighbors, such as "Name by role" under Public API and +the root-cause rule under Required, and follow the context's Prompting section. + +Decided 2026-11-07 (maintainer, moq-dev/rust2 /quest-complete): + +- Which dropped rules join the shared context? + - ✅ Match existing conventions, patterns, and naming + - ✅ Never pass a flaky test by loosening its assertion + - ✅ Short names under a module namespace + - Rejected: "keep the PR to one problem, no drive-by changes", which clashes with + fixing bugs in the code a PR touches and outdated docs inline + - Rejected: "if a skill's usage is unclear, offer to update it" +- Where does this quest go? + - ✅ m0 (recommended) + - m1 + - m2