docs(agents): add working principles and fix stale paths - #893
Open
lukecotter wants to merge 3 commits into
Open
docs(agents): add working principles and fix stale paths#893lukecotter wants to merge 3 commits into
lukecotter wants to merge 3 commits into
Conversation
Add a How to work section covering Think Before Coding, Simplicity First, Surgical Changes and Goal-Driven Execution, each anchored to this repo. Also correct the instructions that had drifted: apex-log-parser was missing from the structure list, and the log-viewer rules pointed at a parser directory that no longer exists. Move the flame chart module doc to AGENTS.md so every tool discovers it, leaving a CLAUDE.md import stub.
"One concern per commit. Show the message and file list, then wait for approval." is a personal working agreement with an assistant, not a property of this repo. It has no meaning for a contributor committing to their own branch.
lukecotter
marked this pull request as ready for review
August 3, 2026 14:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds working practices to the agent instructions, and fixes places where they had drifted from the code.
Working practices
New
## How to worksection inAGENTS.md, four principles each anchored to something real in this repo rather than stated generically:lana/<->log-viewer/boundary.!importantand 1px nudges.log-viewerperformance budgets are part of the acceptance bar.Drift fixed
apex-log-parser/was missing from the monorepo structure, despite being a real fourth TS package imported by 69 files..claude/rules/log-viewer.mdpointed the parser atlog-viewer/src/core/log-parser/, which no longer exists. Repointed atapex-log-parser/src/, and added the paths that carry the real complexity (messaging boundary, tabulator layer, shared components).pnpm lintalready includestsc -b, and the scopedjest --selectProjectsform that CI uses.Flame chart module doc
log-viewer/src/features/timeline/optimised/CLAUDE.mdis renamed toAGENTS.mdso every tool discovers it, with a smallCLAUDE.mdimport stub left behind — mirroring the root layout. Its deadcore/log-parserimport example is corrected.Docs only, no source changes.