From 87fc8e2eee8915cd610c95fd4d8200ddc179bcc5 Mon Sep 17 00:00:00 2001 From: HosungYou <136484001+HosungYou@users.noreply.github.com> Date: Thu, 8 Oct 2026 10:29:09 +0900 Subject: [PATCH] Remove duplicate TypeScript checks and validation instructions --- .github/workflows/ci.yml | 3 --- AGENTS.md | 9 --------- README.md | 8 +++++--- docs/PANEL-WORKER-BRIDGE-VERIFICATION.md | 9 ++------- docs/RELEASE-PROCESS.md | 11 ++++++----- package.json | 2 +- 6 files changed, 14 insertions(+), 28 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index a4ee6ff9..51f8363c 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -25,8 +25,5 @@ jobs: - name: Build run: npm run build - - name: Typecheck - run: npm run typecheck - - name: Pack dry run run: npm run pack:check diff --git a/AGENTS.md b/AGENTS.md index 82f43fcb..7234ff82 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -59,12 +59,3 @@ This folder is for architecture and refactoring design. Do not treat it as the p - For factual, current, or external claims, provide source links or file references when possible. - If a statement cannot be sourced, label it as an inference or estimate instead of presenting it as a fact. - Do not expose internal tool logs, file-search traces, or process commentary in the researcher-facing answer. - -## Scope - -- Project: LongTable -- Current goal: Review LongTable checkpoint behavior, version alignment, and workspace policy. -- Current blocker: Verify whether Researcher Checkpoints trigger correctly after the latest version update. -- Requested perspectives: reviewer, theory_critic, methods_critic, measurement_auditor, ethics_reviewer, voice_keeper. -- Disagreement visibility: show_on_conflict. -- These instructions apply to this directory and its children. diff --git a/README.md b/README.md index 65093af8..7ddf8bb2 100644 --- a/README.md +++ b/README.md @@ -226,12 +226,14 @@ with explicit assumptions instead of interrupting. ```bash npm ci -npm run build npm run test -npm run release:check ``` -Useful smoke checks: +`test` builds the workspaces (including TypeScript checks) and runs the smoke +checks. For a focused change, run the affected workspace build and smoke check. +Use `release:check` once for a release; it includes `test` and the pack dry run. + +Focused smoke checks: ```bash npm run smoke:setup diff --git a/docs/PANEL-WORKER-BRIDGE-VERIFICATION.md b/docs/PANEL-WORKER-BRIDGE-VERIFICATION.md index 7d5533a5..43b5ca44 100644 --- a/docs/PANEL-WORKER-BRIDGE-VERIFICATION.md +++ b/docs/PANEL-WORKER-BRIDGE-VERIFICATION.md @@ -14,12 +14,7 @@ This note records the verification lane for the visible writable panel worker br Run from the repository root after implementation lanes are integrated: ```bash -npm run build -npm run smoke:panel-workers -npm run smoke:panel-handoff -npm run typecheck -npm run test -npm run pack:check +npm run release:check ``` ## Evidence expectations @@ -41,7 +36,7 @@ npm run pack:check - panel skills retain the normalized `PanelResult` contract; - hidden reasoning and raw tool log fields are stripped from recorded state; - generated handoff includes provider-neutral continuation guidance and unincorporated evidence. -- `build`, `typecheck`, `test`, and `pack:check` must pass before the bridge is considered integration-ready. +- `release:check` includes the build, panel smoke checks, and pack dry run required for integration. ## Worker-4 run result diff --git a/docs/RELEASE-PROCESS.md b/docs/RELEASE-PROCESS.md index 6e8f16b2..653dd184 100644 --- a/docs/RELEASE-PROCESS.md +++ b/docs/RELEASE-PROCESS.md @@ -22,13 +22,14 @@ npm run release:check `release:check` runs: -- workspace typecheck -- workspace build +- workspace build, including TypeScript checks +- behavioral smoke checks - npm pack dry-run for publishable workspaces -The root build and typecheck scripts intentionally run workspaces in dependency -order. Do not switch them back to generic `--workspaces` ordering unless package -references are changed to source-level project references. +The root build runs workspaces in dependency order. Do not switch it to generic +`--workspaces` ordering unless package references are changed to source-level +project references. `typecheck` is an alternative when emitted output is not +needed; do not run it again after a successful build of the same source. After publishing a version manually, run `npm ci` again before merging or tagging. This catches lockfile entries whose `integrity` values were generated diff --git a/package.json b/package.json index 91586f3e..19e52ae8 100644 --- a/package.json +++ b/package.json @@ -21,7 +21,7 @@ "smoke:grilling-interview": "node scripts/e2e-grilling-interview-flow.mjs", "smoke:search": "node scripts/smoke-search.mjs", "smoke:scholar-research": "node scripts/smoke-scholar-research.mjs", - "test": "npm run build && npm run smoke:setup && npm run smoke:checkpoints && npm run smoke:hooks && npm run smoke:panel-handoff && npm run smoke:panel-workers && npm run smoke:research-spec-audit && npm run smoke:question-audit && npm run smoke:role-audit && npm run smoke:human-persona && npm run smoke:grilling-interview && npm run smoke:search && npm run smoke:scholar-research && npm run smoke:research && npm run typecheck", + "test": "npm run build && npm run smoke:setup && npm run smoke:checkpoints && npm run smoke:hooks && npm run smoke:panel-handoff && npm run smoke:panel-workers && npm run smoke:research-spec-audit && npm run smoke:question-audit && npm run smoke:role-audit && npm run smoke:human-persona && npm run smoke:grilling-interview && npm run smoke:search && npm run smoke:scholar-research && npm run smoke:research", "pack:check": "npm pack --workspaces --dry-run", "release:check": "npm run test && npm run pack:check", "smoke:research": "node scripts/smoke-research.mjs",