default
> `./episodic_memory.db` unless the test isolates it. A test module that
> exercises audited code (alerts review, sweeps) without monkeypatching
> `openexecutive.audit.log_event` (or the audit logger's DB) leaks rows that
5.8k50d agoReviewMountCreate a free account to mount
this layer owns the logic, with a code trace from producer to consumer.
### Code Style and Review Conventions
Recurring review feedback distilled into rules — following them avoids review round-trips
5.8k4mo agoReviewMountCreate a free account to mount
Amend
A fresh agent reviews the commit as if it were a real codereviewer: checks correctness, style, edge cases, test coverage, and regressions. It also reviews all documentation changes
5.8k4mo agoReviewMountCreate a free account to mount
Research** → Use knowledge base (see RAG workflow below)
4. **Implement** → Write code based on research
5. **Review** → `manage_task("update", task_id="...", status="review")`
6. **Next Task** → `find_tasks(filter
5.8k17mo agoReviewMountCreate a free account to mount
drafts" / the user is about to approve Outbox emails** → Offer to review them, since Sellbird runs on Claude Code and you can do what a scan can't. A draft
5.7k9d agoReviewMountCreate a free account to mount
Before opening a PR or making a large commit**, always run `/simplify` to review changed code for reuse opportunities, quality issues, and efficiency improvements. Fix any issues it finds before
5.6k56d agoReviewMountCreate a free account to mount
template repos. Failure indicates breaking changes that need template updates.
## Special Workflows
### claude.yml
AI codereview integration. Triggered by `@claude` mentions in issues/PRs. Gated to internal team members via `teams.yml
5.6k56d agoReviewMountCreate a free account to mount
file paths in `.expected` files
- **Line numbers**: Critical for matching expected warnings to actual code lines
- **Issue types**: Must match exactly (e.g., `PhanTypeInvalidCallableMethodName`)
- **Error messages**: Complete message text must match
Steady4 mentions in 2 weeks
No other project uses it yet
5.6k13mo agoReviewMountCreate a free account to mount
summary and link to the PR
### Investigating Issues
For older issues, review the state of the code when the issue was raised - it may have already been addressed:
```bash
5.5k33d agoReviewMountCreate a free account to mount
assessment, known concerns, prior review findings, or another reviewer's conclusions. Reviewers must independently infer how the code attempts to meet the stated requirements.
5. Reviewers are read-only. Require
5.5k3d agoReviewMountCreate a free account to mount
reasoning from the code alone.
- When debugging a failure, start from the incident or issue Polylane recorded: it carries the evidence an investigation already gathered.
- Polylane reviews pull requests
5.5k38d agoReviewMountCreate a free account to mount
this repository.
## Communication Style
- Be professional, concise, and direct
- Do NOT use emojis in codereviews, changelogs, or any generated content. You may use professional visual indicators or favor markdown
5.4k4mo agoReviewMountCreate a free account to mount
becomes the CHANGELOG entry)
```
This writes a `.changeset/ .md` file — commit it alongside your code. **You never hand-edit `CHANGELOG.md`; Changesets generates it.** If a PR genuinely needs no release
5.4k4mo agoReviewMountCreate a free account to mount
spaces?") — `features/instance-policy/` (`deriveInstancePolicy` in `utils.ts` is the inventory; `getInstancePolicy` from `data.ts` for server code, `loadInstancePolicy` from `loaders.ts` for pages, `useInstancePolicy` from `client.tsx`). When the organization layer lands this becomes
No other project uses it yet
5.4k9mo agoReviewMountCreate a free account to mount
writing code in that area.
| Document | Read it when |
| --- | --- |
| [docs/conventions/constants.md](docs/conventions/constants.md) | Adding a threshold, interval, ratio, storage key, or any value used twice |
| [docs/conventions/contributing-flow.md](docs/conventions/contributing-flow.md) | Opening, describing, or reviewing
5.3k10mo agoReviewMountCreate a free account to mount
requesting review.
3. Resolve all review comments before merging — do not dismiss without explanation.
---
## CodeReview Standards
### What to Check
1. **Correctness**: Does the code do what it claims
5.2k3mo agoReviewMountCreate a free account to mount
operations; categories alone do not skip tests when running the full suite locally.
### Code Style
```bash
# Check formatting (from src/ directory)
dotnet format --verify-no-changes --severity info --no-restore
5.2k18mo agoReviewMountCreate a free account to mount
A file Claude Code reads at the start of every session. It holds the commands, conventions and warnings the agent needs for this project.
Where does it go?
At the repository root. Claude Code also reads CLAUDE.md files in subdirectories when it works there.
What should it contain?
Build and test commands, the project's layout, conventions that aren't obvious from the code, and mistakes to avoid. Short files tend to work better than long ones.
CLAUDE.md or AGENTS.md?
Claude Code reads CLAUDE.md; most other agents read AGENTS.md. Many projects keep one and point the other at it.