Skip to content

Commit 9ba2460

Browse files
committed
agent stuff
1 parent cdb7e30 commit 9ba2460

45 files changed

Lines changed: 88 additions & 85 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.agents/commands/mind-sets/audit.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,7 @@ Assign a score from 1 (critical) to 5 (excellent) for each area:
348348

349349
## Workflow
350350

351-
1. **Gather context** -- Use AskUserQuestion to collect missing project information
351+
1. **Gather context** -- ask the user to provide missing project information
352352
2. **Read documentation** -- Analyze all provided files and configurations
353353
3. **Load foundations** -- Read `project-foundations-{language}.md` for the project's primary language
354354
4. **Execute audit** -- Follow the 9-section structure systematically

.agents/commands/x-audit.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,13 @@ say so explicitly and restart from STEP 0.
3232

3333
## Your task
3434

35-
- Read `.claude/commands/mind-sets/audit.md` (the audit methodology)
36-
- Read `.claude/commands/mind-sets/project-foundations-cpp.md` (the 16 C++ foundations the audit checks against -- it `$include`s `project-foundations.md`, read that too if referenced)
35+
- Read `.agents/commands/mind-sets/audit.md` (the audit methodology)
36+
- Read `.agents/commands/mind-sets/project-foundations-cpp.md` (the 16 C++ foundations the audit checks against -- it `$include`s `project-foundations.md`, read that too if referenced)
3737
- If a Memory Bank exists in this repo, read it first for project context; if not, skip -- this command does not require one
38-
- **Optional project settings:** if `.claude/commands/cpp-project.md` exists, read it and use its `@build` / `@test` / `@lint` / `@code-nav` values instead of the generic CMake/CTest commands
38+
- **Optional project settings:** if `.agents/commands/cpp-project.md` exists, read it and use its `@build` / `@test` / `@lint` / `@code-nav` values instead of the generic CMake/CTest commands
3939
- **Evidence Discipline (no false negatives):** follow the "Evidence Discipline" section of audit.md -- use the language server / code intelligence (LSAI, clangd) not a single grep; scope to OWN code (exclude vendored / `_deps` / `.cpm_cache` / generated trees -- they inflate counts); NEVER emit a `Missing`/`Absent` verdict without running and quoting the targeted negative check (tests: `ctest -N`, `gtest_discover_tests` / `catch_discover_tests`, `enable_testing` + `*/test*/` targets); report a have/total ratio (e.g. "8/36 libs have tests") as `Partial`, never `Missing`
4040
- **Code-level diagnostics (use LSAI to find problems IN the code, not just the build):** run `mcp__lsai__lsai_diagnostics` (minSeverity warning) and scope the result to OWN code (exclude vendored / SDK / generated / known-rotten trees). Discard the `-Wc++98-compat` / `-Wpre-c++NN-compat` pedantry the analyzer emits under its near-`-Weverything` profile, then report the SUBSTANTIVE warnings -- sign/signedness conversion, field/local shadowing, float `==`/`!=`, unsafe-buffer-usage / pointer arithmetic, unhandled `switch` enums, `const`-dropping casts, unused includes -- as a "Code-Level Diagnostics" finding with `file:line`. Tie any defect the real build hides (warnings-as-errors not on that compiler) back to Foundation #1.
41-
- Use AskUserQuestion to gather project context if not provided (what the project is, primary build system, target platforms, what worries the user)
41+
- Ask the user for project context if not provided (what the project is, primary build system, target platforms, what worries the user)
4242
- Confirm activation with "# Architecture Audit active (C++)"
4343

4444
## Scope reminder (C++ specifics)

.agents/commands/x-debug.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ say so explicitly and restart from STEP 0.
3232
- Skip the BASE_RULE and I will kick your honest ass straight back to STEP 0. Stop, reset your head, and do it right -- anything you produce otherwise is invalid.
3333

3434
## Prerequisites
35-
- Read `.claude/commands/mind-sets/debug-protocol.md` (Protocol D -- the methodology, single source of truth for the rules)
35+
- Read `.agents/commands/mind-sets/debug-protocol.md` (Protocol D -- the methodology, single source of truth for the rules)
3636
- If a Memory Bank exists, read it for context; otherwise skip (not required)
37-
- **Optional project settings:** if `.claude/commands/cpp-project.md` exists, read it and use its `@build` / `@test` / `@asan` / `@code-nav` / `@logs` / `@platform` values instead of the generic commands below
37+
- **Optional project settings:** if `.agents/commands/cpp-project.md` exists, read it and use its `@build` / `@test` / `@asan` / `@code-nav` / `@logs` / `@platform` values instead of the generic commands below
3838

3939
## Purpose
4040

.agents/commands/x-develop.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@ say so explicitly and restart from STEP 0.
3232
- Skip the BASE_RULE and I will kick your honest ass straight back to STEP 0. Stop, reset your head, and do it right -- anything you produce otherwise is invalid.
3333

3434
## Prerequisites
35-
- Read `.claude/commands/mind-sets/cpp-senior.md` (the C++ senior mindset -- the rules you code by)
36-
- Read `.claude/commands/mind-sets/common-coding-style.md` (cross-language rules: immutability, KISS/DRY/YAGNI, file limits)
37-
- Read `.claude/commands/mind-sets/project-foundations-cpp.md` (the foundations the code must satisfy)
38-
- If the methodology is TDDAB: read `.claude/commands/mind-sets/cpp-tddab-overlay.md`
35+
- Read `.agents/commands/mind-sets/cpp-senior.md` (the C++ senior mindset -- the rules you code by)
36+
- Read `.agents/commands/mind-sets/common-coding-style.md` (cross-language rules: immutability, KISS/DRY/YAGNI, file limits)
37+
- Read `.agents/commands/mind-sets/project-foundations-cpp.md` (the foundations the code must satisfy)
38+
- If the methodology is TDDAB: read `.agents/commands/mind-sets/cpp-tddab-overlay.md`
3939
- If a Memory Bank / plan file exists, read it for context; otherwise work from the plan the user points you to
40-
- **Optional project settings:** if `.claude/commands/cpp-project.md` exists, read it and use its `@build` / `@configure` / `@test` / `@lint` / `@asan` values in the per-block VERIFY loop instead of the generic CMake/CTest commands
40+
- **Optional project settings:** if `.agents/commands/cpp-project.md` exists, read it and use its `@build` / `@configure` / `@test` / `@lint` / `@asan` values in the per-block VERIFY loop instead of the generic CMake/CTest commands
4141

4242
## Purpose
4343

@@ -84,4 +84,4 @@ non-trivial.
8484
- **Follow existing patterns and naming** in the target codebase
8585
- **Fix the cause, not the symptom** -- a defensive check that hides the root is rejected
8686
- **One block at a time** -- never batch blocks past a red build
87-
- **Ask when unsure** -- use AskUserQuestion rather than guessing the intent
87+
- **Ask when unsure** -- ask the user rather than guessing the intent

.agents/commands/x-review-plan.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ say so explicitly and restart from STEP 0.
3232
- Skip the BASE_RULE and I will kick your honest ass straight back to STEP 0. Stop, reset your head, and do it right -- anything you produce otherwise is invalid.
3333

3434
## Prerequisites
35-
- Read `.claude/commands/mind-sets/tddab-planner.md` + `.claude/commands/mind-sets/cpp-tddab-overlay.md` (the rules the plan must satisfy)
36-
- Read `.claude/commands/mind-sets/project-foundations-cpp.md` (project conformity)
35+
- Read `.agents/commands/mind-sets/tddab-planner.md` + `.agents/commands/mind-sets/cpp-tddab-overlay.md` (the rules the plan must satisfy)
36+
- Read `.agents/commands/mind-sets/project-foundations-cpp.md` (project conformity)
3737
- If a Memory Bank exists, read it for context; otherwise skip
38-
- **Optional project settings:** if `.claude/commands/cpp-project.md` exists, read it and use its `@code-nav` / `@build` / `@test` values where relevant
38+
- **Optional project settings:** if `.agents/commands/cpp-project.md` exists, read it and use its `@code-nav` / `@build` / `@test` values where relevant
3939

4040
## Steps
4141

.agents/commands/x-tddab.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,11 @@ say so explicitly and restart from STEP 0.
3232
- Skip the BASE_RULE and I will kick your honest ass straight back to STEP 0. Stop, reset your head, and do it right -- anything you produce otherwise is invalid.
3333

3434
## Prerequisites
35-
- Read `.claude/commands/mind-sets/tddab-planner.md` (the base TDDAB methodology -- single source of truth for plan structure)
36-
- Read `.claude/commands/mind-sets/cpp-tddab-overlay.md` (C++ specifics: Catch2/CTest, expected/optional, sanitizers in VERIFY)
37-
- Read `.claude/commands/mind-sets/project-foundations-cpp.md` (the foundations the plan must respect)
35+
- Read `.agents/commands/mind-sets/tddab-planner.md` (the base TDDAB methodology -- single source of truth for plan structure)
36+
- Read `.agents/commands/mind-sets/cpp-tddab-overlay.md` (C++ specifics: Catch2/CTest, expected/optional, sanitizers in VERIFY)
37+
- Read `.agents/commands/mind-sets/project-foundations-cpp.md` (the foundations the plan must respect)
3838
- If a Memory Bank exists, read it for context; otherwise skip
39-
- **Optional project settings:** if `.claude/commands/cpp-project.md` exists, read it and use its `@build` / `@test` / `@asan` values in the VERIFY steps of the plan
39+
- **Optional project settings:** if `.agents/commands/cpp-project.md` exists, read it and use its `@build` / `@test` / `@asan` values in the VERIFY steps of the plan
4040

4141
## Purpose
4242

@@ -47,7 +47,7 @@ context -- no dependency on a later block.
4747

4848
## Steps
4949

50-
1. **Clarify the feature** -- use AskUserQuestion for anything ambiguous (scope,
50+
1. **Clarify the feature** -- ask the user about anything ambiguous (scope,
5151
target API, where it plugs into the existing code).
5252
2. **Decompose bottom-up** -- list the smallest provable units. For C++ that is
5353
usually: value types/DTOs -> domain functions -> the service/composition.

.agents/skills/erhe-cpp-debugging/SKILL.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: erhe-cpp-debugging
3-
description: Debug an erhe C++ crash, abort, VERIFY/ERHE_FATAL, thrown exception, hang or deadlock at the source level with a real callstack and live variable values, instead of guessing from logs/log.txt and a stripped backtrace. On macOS drive lldb from the Bash tool (preferred: the embedded Python SB API emitting JSON); on Windows drive the live Visual Studio via the visualstudio MCP server (breakpoints, launch-under-debugger, step, callstack, locals). Use this for "editor.exe exited non-zero / [crash] unhandled exception / a minidump appeared / a VERIFY or FATAL fired / a C++ exception 0xe06d7363 / it deadlocked or hung" -- the minidump and log are a starting hypothesis, the live callstack + locals are the diagnosis. For GPU "renders wrong" bugs use erhe-renderdoc-gpu-debug instead; this skill is for CPU-side control flow and state.
3+
description: Debug an erhe C++ crash, abort, VERIFY/ERHE_FATAL, thrown exception, hang or deadlock at the source level with a real callstack and live variable values, instead of guessing from logs/log.txt and a stripped backtrace. On macOS drive lldb from a shell (preferred: the embedded Python SB API emitting JSON); on Windows drive the live Visual Studio via the visualstudio MCP server (breakpoints, launch-under-debugger, step, callstack, locals). Use this for "editor.exe exited non-zero / [crash] unhandled exception / a minidump appeared / a VERIFY or FATAL fired / a C++ exception 0xe06d7363 / it deadlocked or hung" -- the minidump and log are a starting hypothesis, the live callstack + locals are the diagnosis. For GPU "renders wrong" bugs use erhe-renderdoc-gpu-debug instead; this skill is for CPU-side control flow and state.
44
---
55

66
# erhe C++ debugging (lldb on macOS / Linux, Visual Studio MCP on Windows)
@@ -11,20 +11,20 @@ variable values at the fault site. Do NOT try to reason a crash out from
1111
`logs/editor_crash_*.dmp` minidump -- they are a hypothesis, not the diagnosis.
1212

1313
This is the authoritative macOS/Linux lldb run-book. The Windows VS-MCP prose
14-
(full tool list, typical flow, gotchas) lives in CLAUDE.md under "Debugging on
14+
(full tool list, typical flow, gotchas) lives in AGENTS.md under "Debugging on
1515
Windows / MSVC (Visual Studio MCP server)".
1616

1717
## Pick the platform path
1818

19-
- **macOS / Linux -> lldb via the Bash tool.** `/usr/bin/lldb` ships with the
19+
- **macOS / Linux -> lldb via a shell.** `/usr/bin/lldb` ships with the
2020
Command Line Tools (no install). All three Debug builds have full symbols:
2121
`build_xcode_{metal,opengl,vulkan}/src/editor/Debug/editor`.
2222
- **Windows -> the `visualstudio` MCP server** (`mcp__visualstudio__*`). It drives a
2323
live VS instance: `debugger_add_breakpoint` -> `debugger_launch` -> poll
2424
`debugger_status` until it breaks -> `debugger_get_callstack` /
2525
`debugger_get_locals` / `debugger_evaluate` -> `debugger_stop`. Always
2626
`solution_info` first to confirm which solution is loaded; for headless-only
27-
repros open `build_vs2026_vulkan_headless/erhe.slnx`. (CLAUDE.md has the full
27+
repros open `build_vs2026_vulkan_headless/erhe.slnx`. (AGENTS.md has the full
2828
tool list and flow.) `lldb-mi`/GDB-MI do not exist here; on Windows use VS-MCP.
2929

3030
## macOS: which lldb interface (most efficient first)

.agents/skills/erhe-headless-verify/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ description: Verify editor behavior end-to-end without a display -- build the he
66
# erhe headless verification (headless Vulkan build + in-editor MCP)
77

88
The condensed run-book for the loop used to verify editor changes headlessly.
9-
Full reference: CLAUDE.md "In-editor MCP server".
9+
Full reference: AGENTS.md "In-editor MCP server".
1010

1111
## Step 1 -- build
1212

@@ -84,7 +84,7 @@ py -3 scripts/mcp_call.py get_scene_nodes b64:<base64-of-{"scene_name":"Default
8484
`transform_selection` it, then re-screenshot and diff by eye.
8585
- Missing capability? Add a new MCP tool to
8686
`src/editor/mcp/mcp_server.{hpp,cpp}` rather than working around it --
87-
CLAUDE.md calls this out as first-class debugging infrastructure.
87+
AGENTS.md calls this out as first-class debugging infrastructure.
8888

8989
## Step 4 -- clean up (always)
9090

.agents/skills/erhe-quest-launch/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ Capture files land in `logs/quest_<YYYYMMDD>_<HHMMSS>.log` (gitignored) and are
5757
The script enforces these invariants -- understand them before changing it:
5858

5959
- **One extra process, ever.** The single backgrounded `adb logcat` is the only process owned by this workflow. Re-running `quest_logcat.sh` kills the previous streamer before starting a new one; `stop` kills without starting a replacement. The shared `adb` server daemon is not "ours" -- it persists across sessions regardless.
60-
- **No leaks across Claude Code sessions.** The PID is tracked in `logs/.logcat.pid` (a real file in the repo, gitignored), so even a fresh shell finds and stops the prior capture.
60+
- **No leaks across AI client sessions.** The PID is tracked in `logs/.logcat.pid` (a real file in the repo, gitignored), so even a fresh shell finds and stops the prior capture.
6161
- **No PID-recycle hazard.** Before killing, the script confirms the recorded PID still points at an `adb` process via `ps -p PID`; if the PID was recycled to something unrelated, it leaves the new owner alone and just removes the stale PID file. Git Bash's `ps` reports the executable path only (no argv), so it matches on `adb` -- fine because every other adb call in this workflow is short-lived.
6262

6363
## Filtering a capture to erhe-only lines

.agents/skills/erhe-renderdoc-gpu-debug/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -225,5 +225,5 @@ rect) settle it without any CPU-readback ambiguity.
225225
pre-tonemap) -- read the **stats**, do not judge brightness by the PNG. The ID
226226
textures are RGBA8/UNORM, so their PNGs are literal.
227227
- `save_texture` path: forward slashes, parent dir must exist, `overwrite:true`.
228-
- The proxy leaves `qrenderdoc` running across Claude Code sessions;
228+
- The proxy leaves `qrenderdoc` running across AI client sessions;
229229
`renderdoc_shutdown` only kills an instance the proxy itself launched.

0 commit comments

Comments
 (0)