Remove checked-in shell completion files

.github/workflows/ci.yml

@@ -45,17 +45,6 @@ jobs:
       - name: Run TypeScript unit tests
         run: bun run test
 
-      - name: Check remaining shell syntax
-        run: bash -n completions/rootcell.bash
-
-      - name: Install ShellCheck
-        run: |
-          sudo apt-get update
-          sudo apt-get install -y shellcheck
-
-      - name: Run ShellCheck
-        run: shellcheck --severity=warning completions/rootcell.bash
-
       - name: Compile Python modules
         run: python3 -m compileall proxy
 

README.md

@@ -339,7 +339,6 @@ proxy/                   allowlists and mitmproxy/dnsmasq firewall code
   agent_spy.py           Bedrock Runtime formatter for `./rootcell spy`
   agent_spy_tui.py       Textual browser for `./rootcell spy --tui`
 pi/agent/                global pi instructions, skills, and extensions
-completions/             bash and zsh completion for `rootcell`
 ```
 
 ## VM Lifecycle
@@ -423,11 +422,9 @@ Do not put provider keys in `home.nix`; the Nix store is world-readable.
 
 ### Shell Completions
 
-`rootcell completion` prints the yargs-generated completion script. The checked-in
-files under `completions/` are generated from that command; refresh them with
-`bun run completions` after changing commands or options. The generated scripts
-register `rootcell`, so put `rootcell` on `PATH` before sourcing or installing
-them.
+`rootcell completion` prints the yargs-generated completion script. Generate it
+from the installed `rootcell` command so completions stay in sync with the
+version on `PATH`.
 
 For zsh, after `compinit`:
 

package.json

@@ -3,7 +3,6 @@
   "private": true,
   "type": "module",
   "scripts": {
-    "completions": "SHELL=/bin/bash ./rootcell completion | sed '${/^$/d;}' > completions/rootcell.bash && SHELL=/bin/zsh ./rootcell completion | sed '${/^$/d;}' > completions/rootcell.zsh",
     "typecheck": "tsc --noEmit",
     "lint": "eslint \"src/**/*.{ts,js,mjs,cjs}\" eslint.config.ts vitest.config.ts",
     "test": "vitest --project unit --run",

src/rootcell/integration/common/fixtures.ts

@@ -20,7 +20,7 @@ export const defaultSpyOptions = {
 export function findRepoDir(importMetaUrl: string): string {
   let dir = dirname(resolve(fileURLToPath(importMetaUrl)));
   for (;;) {
-    if (existsSync(resolve(dir, "flake.nix")) && existsSync(resolve(dir, "completions"))) {
+    if (existsSync(resolve(dir, "flake.nix")) && existsSync(resolve(dir, "src/rootcell"))) {
       return dir;
     }
     const parent = dirname(dir);

src/rootcell/rootcell.test.ts

@@ -1131,14 +1131,14 @@ describe("reload helper", () => {
   });
 });
 
-describe("completion files", () => {
+describe("shell completions", () => {
   test("bash and zsh completions are generated by yargs", () => {
-    const bash = readFileSync("completions/rootcell.bash", "utf8");
-    const zsh = readFileSync("completions/rootcell.zsh", "utf8");
-    expect(bash).toBe(generatedCompletion("/bin/bash"));
-    expect(zsh).toBe(generatedCompletion("/bin/zsh"));
+    const bash = generatedCompletion("/bin/bash");
+    const zsh = generatedCompletion("/bin/zsh");
     expect(bash).toContain("yargs command completion script");
     expect(zsh).toContain("yargs command completion script");
+    expect(bash).toContain("complete -o bashdefault -o default -F _rootcell_yargs_completions rootcell");
+    expect(zsh).toContain("compdef _rootcell_yargs_completions rootcell");
   });
 
   test("yargs completion API includes all typed subcommands", () => {

src/rootcell/rootcell.ts

@@ -84,7 +84,7 @@ function shellQuote(value: string): string {
 function repoDirFromImportMeta(importMetaPath: string): string {
   let dir = dirname(resolve(importMetaPath));
   for (;;) {
-    if (existsSync(join(dir, "flake.nix")) && existsSync(join(dir, "completions"))) {
+    if (existsSync(join(dir, "flake.nix")) && existsSync(join(dir, "src/rootcell"))) {
       return dir;
     }
     const parent = dirname(dir);