anything touching behavior, codegen, or a public contract gets the build and the relevant tests before you call it done. A comment or doc fix rarely needs a build, though
No other project uses it yet
18k2y agoReviewMountCreate a free account to mount
downloads that may fail in restricted environments. Document as "network-dependent" if install fails.
### Testing and Validation
#### Unit Tests (limited due to environment requirements)
```bash
cd lifecycle
go test ./pkg/utils
No other project uses it yet
18k3y agoReviewMountCreate a free account to mount
Setup commands
- Install deps: `bun install`
- Start dev server: `bun run dev`
- Run tests: `bun run test`
## Code style
- TypeScript strict mode
- Single quotes, no semicolons
- Use functional patterns where
No other project uses it yet
18k2y agoReviewMountCreate a free account to mount
when working with code in this repository.
## Codebase Overview
Vitest is a next-generation testing framework powered by Vite. This is a monorepo using pnpm workspaces.
## Essential references
- Agent-specific
No other project uses it yet
17k48d agoReviewMountCreate a free account to mount
writing a new one.
## Verification Scripts
- Do not create verification scripts or tinker when tests cover that functionality and prove it works. Unit and feature tests are more important.
## Application
Steady2 mentions in 2 weeks
No other project uses it yet
17k6mo agoReviewMountCreate a free account to mount
main codebase - follows Google C++ Style Guide with modifications)
- Python 3 (management tools, tests)
- CMake (build system using Ninja)
- Bash (scripts and utilities)
**Key Components:**
- **MON (Monitor)**: Cluster membership
No other project uses it yet
17k18d agoReviewMountCreate a free account to mount
violation with the rule, a bug with the
input that triggers it, a missing test with the untested behavior. Skip praise and summaries.
- Do not request changes for anything
No other project uses it yet
17k6mo agoReviewMountCreate a free account to mount
prefer using isomorphic code like fetch, ReadableStream, etc. instead of Node.js specific code. Our tests are all vitest. We use prisma in internal-packages/database for our database interactions using
Steady7 mentions in 2 weeks
No other project uses it yet
17k3mo agoReviewMountCreate a free account to mount
architecture, and conventions. These notes cover review.
## Priorities
Report correctness problems first, then missing test coverage, then documentation that contradicts
the code. For correctness findings, identify a failing input
Steady2 mentions in 2 weeks
No other project uses it yet
16k4y agoReviewMountCreate a free account to mount
name working, warn on stderr, document, remove in a later major release).
## Tests as a compatibility signal
1. **Modified expectations.** Explicitly call out any PR that updates, loosens, skips
Steady3 mentions in 2 weeks
No other project uses it yet
16k11mo agoReviewMountCreate a free account to mount
trick"
- Prefer readable code over clever one-liners: name intermediate variables, easier debug
- Integration tests are sweet spot — not unit tests (break on refactor), not e2e (hard debug)
- When
Steady2 mentions in 2 weeks
No other project uses it yet
16k6y agoReviewMountCreate a free account to mount
recommended for development (fastest compilation)
# Build Scylla binary only (sufficient for Python integration tests)
ninja build/ /scylla
# Build specific test
ninja build/ /test/boost/
```
### Using frozen toolchain (Docker)
Prefix any build
No other project uses it yet
16k19mo agoReviewMountCreate a free account to mount
drafts; the coordinator owns the final assessment and explicitly authorized publication.
## Build and Test
```bash
# Install all packages (from repo root)
pnpm install
# Build the core library
cd packages/pyright-internal && pnpm
No other project uses it yet
16k9d agoReviewMountCreate a free account to mount