secret, credential, API key, or customer data is in the code or in the
tests.
- Logging does not include sensitive data: no prompts, no episode content,
no credentials.
- New behavior
No other project uses it yet
32k41d agoReviewMountCreate a free account to mount
graph is cheaper than scanning files and
gives you structural context (callers, dependents, test coverage) that file search cannot.
### When to use graph tools FIRST
- **Exploring code**: `semantic_search_nodes
1 project uses a version of this
32k5mo agoReviewMountCreate a free account to mount
build dir
- Formatting (required before commits):
- `cmake --build . --target format` (runs clang-format)
- Tests:
- Run all tests: `ctest -j ` from build dir
- Run single test (verbose): `ctest -R -V`
- Translations
Steady3 mentions in 2 weeks
No other project uses it yet
30k23mo agoReviewMountCreate a free account to mount
groups required by the project.
- `uv run -p 3.14 --all-groups --all-extras --locked tests/` to run tests on a specific Python version. Please use the `-p` flag often
No other project uses it yet
30k24mo agoReviewMountCreate a free account to mount
remove it.
- **IMPORTANT:** Many bazel targets are very slow/expensive to build/test. Only build or test with
bazel when told to, and prefer the narrowest target that covers the change.
## Toolchain
29k52d agoReviewMountCreate a free account to mount
channels or window lifecycle is ambiguous
- Acknowledge good patterns when you see them
## Testing Standards
- Changes to `src/utils/reader/` (live iframe rendering) require manual verification for layout regressions
- New IPC handlers
No other project uses it yet
29k18d agoReviewMountCreate a free account to mount
files listed in its routing table. Read those, not this file, for workflow,
build, test, and scope rules.
The rules most often missed in review:
- A PR must close
Steady3 mentions in 2 weeks
1 project uses a version of this
28k7mo agoReviewMountCreate a free account to mount
except in the files it still suppresses.
- Wildcard imports in `src/main`. Static wildcards in tests are fine.
- An empty `catch` block. A tolerated exception is logged, or named `ignored
Steady5 mentions in 2 weeks
1 project uses a version of this
27k3mo agoReviewMountCreate a free account to mount
src/PowerShell`) - Microsoft.WinGet.Client and Microsoft.WinGet.Configuration cmdlets
- **Configuration system** - DSC-based system configuration using WinGet
## Building, Testing, and Running
### Initial Setup
Use a configuration file in `.config` as in `winget configure .config/configuration.winget
27k9d agoReviewMountCreate a free account to mount
lint:advisory` (includes warnings omitted by the normal quiet lint commands)
- **Unit tests**: `npm run test:unit`
## Code review
When reviewing a PR or reviewing changes on the current branch
Steady6 mentions in 2 weeks
No other project uses it yet
26k4y agoReviewMountCreate a free account to mount
operator/ # Operator proto
placement/ # Placement service proto
scheduler/ # Scheduler service proto
sentry/ # Sentry proto
tests/
integration/ # Integration tests (run against real daprd binary)
e2e/ # End-to-end tests (require Kubernetes
No other project uses it yet
26k5mo agoReviewMountCreate a free account to mount
development takes place in the `internal` directory, and most behaviors can be tested via compiler tests.
Most development on the codebase is in Go.
Standard Go commands and practices apply
26k9mo agoReviewMountCreate a free account to mount