From 2886b301d62c040aec74deeb7c2da76c1487ce51 Mon Sep 17 00:00:00 2001 From: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com> Date: Fri, 26 Jun 2026 18:28:10 +0000 Subject: [PATCH 1/3] =?UTF-8?q?feat:=20implement=20issue=20#332=20?= =?UTF-8?q?=E2=80=94=20Compliance:=20copilot-instructions-missing-tech-sta?= =?UTF-8?q?ck?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .github/copilot-instructions.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index 4f204a7f..3a721b38 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -3,3 +3,12 @@ All instructions for this repository are maintained in [`AGENTS.md`](../AGENTS.md) at the repository root. Please read and follow the guidance in that file. + +## Tech Stack + +- **Runtime:** Google Apps Script (V8) for deployed code; Node.js for tooling and tests +- **Language:** JavaScript (`.gs` / `.js`), with TypeScript available for type checking (`tsc --noEmit`) +- **Testing:** Jest (unit, `npm test`) · Playwright (e2e, `npm run test:e2e`) +- **Linting:** ESLint + Prettier (`npm run check`) +- **Coverage thresholds:** 100% lines, 95% statements/functions, 85% branches +- **Key libraries:** `@types/google-apps-script` (GAS type defs), `ts-jest`, `husky` + `lint-staged` (pre-commit), `@commitlint` (conventional commits) From 9ce1a86140115249c2141219d8d82b6f7b7f6618 Mon Sep 17 00:00:00 2001 From: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com> Date: Fri, 26 Jun 2026 18:32:05 +0000 Subject: [PATCH 2/3] fix(reviews): address review comments [skip ci-relay] --- .github/copilot-instructions.md | 4 ++-- AGENTS.md | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index 3a721b38..810f8c53 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -7,8 +7,8 @@ Please read and follow the guidance in that file. ## Tech Stack - **Runtime:** Google Apps Script (V8) for deployed code; Node.js for tooling and tests -- **Language:** JavaScript (`.gs` / `.js`), with TypeScript available for type checking (`tsc --noEmit`) +- **Language:** JavaScript (`.gs` / `.js`), with TypeScript available for type checking (`npm run typecheck`) - **Testing:** Jest (unit, `npm test`) · Playwright (e2e, `npm run test:e2e`) - **Linting:** ESLint + Prettier (`npm run check`) -- **Coverage thresholds:** 100% lines, 95% statements/functions, 85% branches +- **Coverage thresholds:** 99% lines, 95% statements/functions, 85% branches - **Key libraries:** `@types/google-apps-script` (GAS type defs), `ts-jest`, `husky` + `lint-staged` (pre-commit), `@commitlint` (conventional commits) diff --git a/AGENTS.md b/AGENTS.md index 3af9de41..fb12866f 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -33,7 +33,7 @@ This file follows the AGENTS.md conventions (see https://agents.md/) and provide ## Tests & CI (repo conventions) -- **Coverage thresholds:** 100% lines, 95% statements/functions, 85% branches. Verify locally with `npm test -- --coverage` (or `npx jest --coverage`) and ensure CI coverage meets these requirements. PRs that reduce coverage below these thresholds will be rejected. +- **Coverage thresholds:** 99% lines, 95% statements/functions, 85% branches. Verify locally with `npm test -- --coverage` (or `npx jest --coverage`) and ensure CI coverage meets these requirements. PRs that reduce coverage below these thresholds will be rejected. - Before requesting review or marking a PR as ready: - Run `npm test` and ensure all tests pass locally - Run `npm test -- --coverage` and ensure coverage meets thresholds From 105e9a935d40ee0085e0f06fd0e9ecddf1c0f722 Mon Sep 17 00:00:00 2001 From: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com> Date: Fri, 26 Jun 2026 18:34:55 +0000 Subject: [PATCH 3/3] chore: apply manual instructions [skip ci-relay] --- CLAUDE.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CLAUDE.md b/CLAUDE.md index 69cb05db..5d97a9fc 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -17,7 +17,7 @@ This file provides Claude Code-specific instructions. For comprehensive agent gu ## Key Rules 1. **TDD is mandatory** — write tests before implementation, include tests in the same PR -2. **Coverage thresholds** — 100% lines, 95% statements/functions, 85% branches +2. **Coverage thresholds** — 99% lines, 95% statements/functions, 85% branches 3. **Avoid coverage-ignore comments and `.skip()`** — only use in rare, unavoidable cases (per AGENTS.md) with clear justification 4. **GAS testing pattern** — extract logic to `src/index.js` with `module.exports`, inject GAS services as parameters 5. **Always run** `npx prettier --write .` before committing — pre-commit hooks don't run in agent sessions