From 920f88b34d47434bb9449a6a7fd7584bdaf6abbc Mon Sep 17 00:00:00 2001 From: fireshaper Date: Fri, 2 Oct 2026 17:34:01 -0400 Subject: [PATCH] Step 1.4: ignore in-workspace pnpm store in CI The Gitea runner places the pnpm store inside the workspace, so prettier --check failed on its files. Co-Authored-By: Claude Opus 5.5 --- .gitignore | 1 + .prettierignore | 1 + DECISIONS.md | 5 +++++ eslint.config.js | 2 +- 4 files changed, 8 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index fd2e43c..90fa285 100644 --- a/.gitignore +++ b/.gitignore @@ -5,3 +5,4 @@ coverage/ .env .env.local .DS_Store +.pnpm-store/ diff --git a/.prettierignore b/.prettierignore index 5b5a19b..51378bf 100644 --- a/.prettierignore +++ b/.prettierignore @@ -1,2 +1,3 @@ pnpm-lock.yaml memory-bank/ +.pnpm-store/ diff --git a/DECISIONS.md b/DECISIONS.md index 4e979ef..c1a5617 100644 --- a/DECISIONS.md +++ b/DECISIONS.md @@ -20,3 +20,8 @@ Implementation decisions not spelled out in `memory-bank/implementation-plan.md` - **Vitest projects**: the root `vitest.config.ts` treats each `apps/*` and `packages/*` folder as a project, so `pnpm test` runs every package's tests. Packages can add their own `vitest.config.ts` when they need a different environment. - **Tests live in each package's `test/` folder** and are included in that package's tsconfig, so `pnpm typecheck` covers them. + +## Step 1.4 + +- **CI runs on Gitea Actions** (`.gitea/workflows/ci.yml`): install with a frozen lockfile, then typecheck, lint and test. +- **`.pnpm-store/` is ignored** by Prettier, ESLint and git. On the Gitea runner, pnpm puts its store inside the workspace, and `prettier --check` failed on the store's files (CI run #1). diff --git a/eslint.config.js b/eslint.config.js index 2b71e81..2226a29 100644 --- a/eslint.config.js +++ b/eslint.config.js @@ -8,7 +8,7 @@ import tseslint from "typescript-eslint"; const impureMessage = "Not allowed in pure packages; pass time and seeds in as arguments."; export default tseslint.config( - { ignores: ["**/node_modules/**", "**/dist/**", "**/coverage/**"] }, + { ignores: ["**/node_modules/**", "**/dist/**", "**/coverage/**", ".pnpm-store/**"] }, js.configs.recommended, ...tseslint.configs.recommended, {