-
-
Notifications
You must be signed in to change notification settings - Fork 0
policy: Bun is tier 1, Deno is being removed — correct local CLAUDE.md #70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
e6857f6
a4a1bd8
c5726c0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -22,15 +22,15 @@ | |
| ### ALLOWED Languages & Tools | ||
|
|
||
| | Language/Tool | Use Case | Notes | | ||
| |---------------|----------|-------| | ||
|
Check failure on line 25 in .claude/CLAUDE.md
|
||
| | **AffineScript** | Primary application code | Affine-typed, compiles to typed-wasm or Deno-ESM | | ||
| | **Deno** | Runtime & package management | Replaces Node/npm/bun | | ||
| | **AffineScript** | Primary application code | Affine-typed, compiles to typed-wasm or ESM | | ||
| | **Bun** | JS runtime & package management (tier 1) | Default for all new work. Runs compiled ESM/JS directly — no bundler step. Uses an npm-compatible `package.json` plus `bun.lock` — both are expected, not anti-patterns. | | ||
| | **Rust** | Performance-critical, systems, WASM | Preferred for CLI tools | | ||
| | **Tauri 2.0+** | Mobile apps (iOS/Android) | Rust backend + web UI | | ||
| | **Dioxus** | Mobile apps (native UI) | Pure Rust, React-like | | ||
| | **Gleam** | Backend services | Runs on BEAM or compiles to JS | | ||
| | **Bash/POSIX Shell** | Scripts, automation | Keep minimal | | ||
| | **JavaScript** | Only where AffineScript cannot | MCP protocol glue, Deno APIs | | ||
| | **JavaScript** | Only where AffineScript cannot | MCP protocol glue, Bun APIs | | ||
| | **Python** | SaltStack only | No other Python permitted | | ||
| | **Nickel** | Configuration language | For complex configs | | ||
| | **Guile Scheme** | State/meta files | STATE.scm, META.scm, ECOSYSTEM.scm | | ||
|
|
@@ -41,12 +41,13 @@ | |
| ### BANNED - Do Not Use | ||
|
|
||
| | Banned | Replacement | | ||
| |--------|-------------| | ||
|
Check failure on line 44 in .claude/CLAUDE.md
|
||
| | TypeScript | AffineScript | | ||
| | Node.js | Deno | | ||
| | npm | Deno | | ||
| | Bun | Deno | | ||
| | pnpm/yarn | Deno | | ||
| | ReScript | AffineScript | | ||
| | Deno | Bun | | ||
| | Node.js | Bun | | ||
| | npm | Bun | | ||
| | pnpm/yarn | Bun | | ||
| | Go | Rust | | ||
| | Python (general) | AffineScript/Rust | | ||
| | Java/Kotlin | Rust/Tauri/Dioxus | | ||
|
|
@@ -66,8 +67,8 @@ | |
| ### Enforcement Rules | ||
|
|
||
| 1. **No new TypeScript files** - Convert existing TS to AffineScript | ||
| 2. **No package.json - use deno.json deps** - Use deno.json imports | ||
| 3. **No node_modules in production** - Deno caches deps automatically | ||
| 2. **Use `package.json` + `bun.lock` for JS runtime deps** - Bun is npm-compatible; a manifest is REQUIRED | ||
| 3. **`bun install --production --frozen-lockfile` for production deps** - resolved from `package.json` and pinned via `bun.lock`; `--frozen-lockfile` makes a lockfile mismatch a build failure rather than a silent re-resolve | ||
| 4. **No Go code** - Use Rust instead | ||
| 5. **Python only for SaltStack** - All other Python must be rewritten | ||
| 6. **No Kotlin/Swift for mobile** - Use Tauri 2.0+ or Dioxus | ||
|
|
@@ -76,7 +77,7 @@ | |
|
|
||
| - **Primary**: Guix (guix.scm) | ||
| - **Fallback**: Nix (flake.nix) | ||
| - **JS deps**: Deno (deno.json imports) | ||
| - **JS deps**: Bun (`package.json` + `bun.lock`). Declare tooling as a devDependency and run `bunx --no-install --bun <tool>` — a bare `bunx <tool>` can fetch an unpinned package and may start Node via its shebang. | ||
|
|
||
| ### Security Requirements | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟡 MEDIUM RISK
Rule #1 does not reflect the ReScript repair mentioned in the PR description. It still refers to 'TypeScript' rather than 'ReScript'.