Fix tests, update instructions

Author: burner1024

AGENTS.md

@@ -9,6 +9,7 @@ This file provides guidance to AI agents (Claude, Gemini, etc.) when working wit
 - **External library packaging:** Libraries imported by transpiler files (ielib, folib) must use **named re-exports** (`export { X } from './module'`), not star re-exports (`export * from './module'`). Ambient declarations (`declare function`, `declare const`) must live in `.d.ts` files, not `.ts` files. See folib's `src/index.ts` for the correct pattern.
 - **URI normalization:** All URIs entering the provider system are normalized via `normalizeUri()` from `core/normalized-uri.ts`. The `ProviderRegistry` handles this at the gateway. If you add new URI-accepting methods to the registry, normalize them. If you use URIs as Map/Set keys elsewhere, use `NormalizedUri` branded type.
 - **User-facing messages:** Never call `connection.window.showInformationMessage/showWarningMessage/showErrorMessage` directly in server code. Use `showInfo()`, `showWarning()`, `showError()`, or `showErrorWithActions()` from `user-messages.ts` — they auto-decode `file://` URIs to readable paths. An oxlint rule enforces this.
+- **Temporary artifacts:** Put transient test/build files under the repo-level `tmp/` directory (or `os.tmpdir()` when system temp is required). Do not create ad hoc temp directories under source trees like `server/test/`, `cli/test/`, or `scripts/**`.
 
 ## Project Overview
 

CONTRIBUTING.md

@@ -34,3 +34,9 @@ Press F5 in VSCode to launch the Extension Development Host. Server attaches on
 Server logs: Output panel, "BGforge MLS" channel.
 
 TS plugin logs: set `"typescript.tsserver.log": "verbose"` in settings, check Output panel under "TypeScript".
+
+## Temporary Files
+
+Keep transient test/build artifacts under the repo-level `tmp/` directory unless a tool specifically requires system temp storage.
+
+Do not create temporary directories inside source or fixture trees such as `server/test/`, `cli/test/`, or `scripts/**`.

cli/test/bin-cli.test.ts

@@ -28,7 +28,7 @@ function run(...args: string[]): { code: number; stdout: string; stderr: string
 }
 
 describe("bin CLI integration", () => {
-    const tmpDir = path.resolve("cli/test/.tmp-bin");
+    const tmpDir = path.resolve("tmp/cli-test-bin");
 
     beforeEach(() => {
         if (!fs.existsSync(CLI)) {

cli/test/cli-utils.test.ts

@@ -100,7 +100,7 @@ describe("safeProcess", () => {
 });
 
 describe("findFiles", () => {
-    const tmpDir = path.join(process.cwd(), "cli/test/.tmp-findfiles");
+    const tmpDir = path.resolve("tmp/cli-test-findfiles");
 
     beforeEach(() => {
         fs.mkdirSync(path.join(tmpDir, "sub"), { recursive: true });
@@ -217,7 +217,7 @@ describe("runCli", () => {
     let exitSpy: ReturnType<typeof vi.spyOn>;
     let logSpy: ReturnType<typeof vi.spyOn>;
     let errorSpy: ReturnType<typeof vi.spyOn>;
-    const tmpDir = path.join(process.cwd(), "cli/test/.tmp-runcli");
+    const tmpDir = path.resolve("tmp/cli-test-runcli");
 
     beforeEach(() => {
         exitSpy = vi.spyOn(process, "exit").mockImplementation(() => { throw new Error("exit"); });

cli/test/format-cli.test.ts

@@ -27,7 +27,7 @@ function run(...args: string[]): { code: number; stdout: string; stderr: string
 }
 
 describe("format CLI integration", () => {
-    const tmpDir = path.resolve("cli/test/.tmp-format");
+    const tmpDir = path.resolve("tmp/cli-test-format");
 
     beforeEach(() => {
         if (!fs.existsSync(CLI)) {

cli/test/transpile-cli.test.ts

@@ -29,7 +29,7 @@ function run(...args: string[]): { code: number; stdout: string; stderr: string
 }
 
 describe("transpile CLI integration", () => {
-    const tmpDir = path.resolve("cli/test/.tmp-transpile");
+    const tmpDir = path.resolve("tmp/cli-test-transpile");
 
     beforeEach(() => {
         if (!fs.existsSync(CLI)) {

knip.ts

@@ -24,15 +24,20 @@ const config: KnipConfig = {
             ],
         },
         server: {
-            // Entry resolved from package.json "main" field (src/server.ts).
-            // No explicit entry needed here.
+            // Point knip at the TypeScript source entry directly.
+            // The package.json "main" field targets the built JS output.
+            entry: ["src/server.ts"],
             // vitest.smoke.config.ts is a separate config run by scripts/test.sh, not imported.
             ignore: [
                 "**/*.d.ts",
                 "vitest.smoke.config.ts",
                 // Created at runtime by enum-transform.test.ts, may exist during parallel Knip runs
                 ...(isProductionKnip
-                    ? ["vitest.integration.config.ts", "test/integration/**", "test/tmp-bundle-test/**"]
+                    ? [
+                        "src/**",
+                        "vitest.integration.config.ts",
+                        "test/integration/**",
+                    ]
                     : []),
             ],
         },

scripts/README.md

@@ -27,6 +27,12 @@ See also: [CONTRIBUTING.md](../CONTRIBUTING.md) | [docs/architecture.md](../docs
 - **Smoke test** (`test/smoke-stdio.test.ts`) -- requires a built server bundle (`pnpm build:base:server`). Run as part of `pnpm test` instead, which builds the bundle first.
 - **Integration tests** (`test/integration/`) -- require external repos cloned via `pnpm test:external`. Run standalone with `cd server && pnpm test:integration`, or as part of `pnpm test` (which clones repos first).
 
+## Temporary Artifacts
+
+Store transient logs, generated scratch files, and test temp directories under the repo-level `tmp/` directory.
+
+Avoid writing temporary data into source and fixture trees such as `server/test/`, `cli/test/`, `scripts/**`, or `grammars/**`.
+
 ## Running individual tests
 
 ```bash

server/test/enum-transform.test.ts

@@ -270,7 +270,7 @@ describe("expandEnumPropertyAccess — externalized enums", () => {
 
 describe("bundle (esbuild integration)", () => {
     // Temp directory for creating ielib-like package structures
-    const tmpDir = path.resolve(__dirname, "tmp-bundle-test");
+    const tmpDir = path.resolve("tmp/server-test-bundle");
 
     function writeTmpFile(relPath: string, content: string): string {
         const filePath = path.join(tmpDir, relPath);

server/test/inline-functions.test.ts

@@ -242,7 +242,7 @@ describe("collectReferencedIdentifiers", () => {
 });
 
 describe("extractInlineFunctionsFromFiles", () => {
-    const tmpDir = path.resolve(__dirname, "tmp-inline-test");
+    const tmpDir = path.resolve("tmp/server-test-inline");
 
     function writeTmpFile(name: string, content: string): string {
         if (!fs.existsSync(tmpDir)) fs.mkdirSync(tmpDir, { recursive: true });