Skip to content

Commit e462d40

Browse files
committed
chore(agent): reduce default repo context and deduplicate feature guidance
1 parent d17c15d commit e462d40

2 files changed

Lines changed: 6 additions & 8 deletions

File tree

.agents/skills/feature-change/SKILL.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ Implement feature changes in the existing architecture without creating drift be
99

1010
# Guardrails
1111

12-
- work in source files, not generated output
1312
- patch the earliest correct layer
1413
- preserve the current separation between domain logic, runtime lifecycle, feature rendering, and config definitions
1514
- use proper German umlauts directly in user-facing wording
@@ -27,7 +26,6 @@ Before editing, identify:
2726
## 2. Keep architecture intact
2827

2928
Do not solve feature problems by:
30-
- hand-editing `dist/`
3129
- duplicating logic across modules
3230
- hiding semantic logic in CSS
3331
- introducing one-off hacks where a shared helper belongs
@@ -44,14 +42,12 @@ Check whether the feature change also needs updates in:
4442
## 4. Protect the behavior
4543

4644
- add or update the closest meaningful tests
47-
- run `npm run check:syntax` before final validation
4845
- leave final validation selection to `$repo-validation`
4946
- leave release packaging to `$userscript-release` when shipped behavior changed
5047

5148
# Output requirements
5249

5350
A valid result from this skill must:
54-
- place the change in source, not generated files
5551
- preserve the project architecture
5652
- include the necessary test updates
5753
- leave the repository ready for validation and release workflows

AGENTS.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@ Priority: truthfulness, correctness, proportional validation.
66

77
- prefer minimal diffs and existing conventions
88
- preserve existing architecture boundaries unless the task explicitly requires changing them
9+
- keep Codex context small: prefer the smallest relevant source files for the task
10+
- do not inspect, quote, or diff `dist/**` unless the task explicitly requires release, build-artifact, packaged-output, or shipped-userscript verification work
11+
- do not inspect, quote, or diff `docs/**` by default; use docs only when the task is documentation, wording sync, feature audit, or explicitly cites a doc file
912
- work in source, not `dist/`
13+
- never hand-edit generated files; refresh them only through the build flow when release work is explicitly requested
1014
- keep standard ESLint coverage scoped to actively maintained source; exclude archive, backup, vendor, and generated trees from the default lint surface
15+
- use `.agents/skills/repo-validation/SKILL.md` after changes to choose and report the smallest sufficient validation
1116
- run `npm run lint` before declaring done when changes touch linted JS/MJS source, tests, loader code, scripts, or lint configuration
12-
- when changes touch Sonar-scanned JS/MJS source or project-level analysis config, treat a SonarQube check as part of the expected validation surface; prefer the SonarQube connection and project data configured in `~/.codex/config.toml`, run it whenever server/auth access is available, and report it explicitly as executed or blocked rather than silently skipping it; treat Sonar as complementary to lint/tests rather than a replacement
13-
- when SonarQube reports new or still-open issues for the current work, fix them in-source and rerun the relevant local validation plus SonarQube in a loop until the touched scope is clean or a concrete blocker is reached; do not stop at a green quality gate if open issues, bugs, vulnerabilities, or code smells for the touched code still remain without explanation
14-
- never hand-edit generated files; refresh them only through the build flow when release work is explicitly requested
15-
- use `.agents/skills/repo-validation/SKILL.md` after changes to choose the smallest sufficient validation
17+
- when changes touch Sonar-scanned JS/MJS source or project-level analysis config, include SonarQube validation whenever server/auth access is available, report it explicitly as executed or blocked, and iterate on fixable findings in the touched scope until clean or concretely blocked
1618
- when SonarQube access, project health, or issue triage is requested, prefer the configured local `sonarqube` MCP server or the SonarQube base URL, project key, and project name from `~/.codex/config.toml`, and re-verify live server status and project visibility instead of assuming an earlier access check is still current
1719
- never copy SonarQube tokens or other local credentials into repo-tracked files; rely on local MCP or environment configuration instead
1820
- when browser access is available, Codex may use the connected Chrome browser for analysis, tests, DOM inspection, console, network checks, and task-focused interactions, but only in the currently active tab; do not use other open tabs, switch tabs automatically, open additional tabs, or navigate outside the active tab unless the task explicitly requires it, keep any browser-side changes minimal and limited to debugging, reproduction, verification, or UI tests, use extra caution before potentially destructive actions, and if browser access is unavailable, continue normally and state what could not be verified

0 commit comments

Comments
 (0)