Commit 1f959cd
committed
policy: address review — drop the .ts contradiction, ban Deno, pin bunx
Review feedback from codacy-production and coderabbitai on the policy wave.
Three substantive points, all accepted:
1. ".ts CONTRADICTION" (codacy, MEDIUM, raised on most of the wave). The Bun row
said "Executes .ts directly, no build step" in a file whose BANNED table bans
TypeScript. OWNER RULING: TypeScript "should not exist at all", so advertising
Bun's TypeScript capability is wrong regardless of whether it is true. Every
.ts reference is removed from the row, including "JS/TS" in its label.
2. "DENO MISSING FROM BANNED" (codacy, raised repeatedly). The wave removed Deno
from ALLOWED but never added it to BANNED, so the ruling was only half
expressed. Added | Deno | Bun |.
3. "UNPINNED bunx" (coderabbitai, Security & Privacy). A bare `bunx <tool>` can
fetch a package outside package.json/bun.lock, and can start Node via a
shebang - both contrary to estate SHA-pinning doctrine and the Node ban.
Guidance now requires a declared devDependency plus
`bunx --no-install --bun <tool>`.
NOT taken: "a npm-compatible" (LanguageTool is wrong, "an" is correct before a
vowel sound); "--frozen-lockfile is redundant" (correct - no change needed, and
none made); the Nix->Guix point (real, but a separate ruling, deliberately not
folded into a Deno/Bun change).1 parent 352f28e commit 1f959cd
1 file changed
Lines changed: 3 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
| 27 | + | |
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
| 45 | + | |
45 | 46 | | |
46 | 47 | | |
47 | 48 | | |
| |||
74 | 75 | | |
75 | 76 | | |
76 | 77 | | |
77 | | - | |
| 78 | + | |
78 | 79 | | |
79 | 80 | | |
80 | 81 | | |
| |||
0 commit comments