format and no run on tag lint test

Author: y3rsh

.cursor/skills/npmjs-publish/SKILL.md

@@ -41,7 +41,7 @@ Do not duplicate version parsing: import **`resolve_version_input`** (or helpers
 `.github/workflows/npmjs-publish.yaml`:
 
 - **PR** (paths): lint + test jobs, no `needs` between them
-- **push** `npmjs-publish@*`: publish job runs preflight with version from `github.ref`
+- **push** `npmjs-publish@*`: **publish** job only (preflight with version from `github.ref`), no lint or unit test jobs
 
 ## Monorepo build order
 

.github/workflows/npmjs-publish.yaml

@@ -1,6 +1,6 @@
 name: npmjs publish
 
-# PR: lint and unit tests only (file-filtered). Tag push: same jobs plus publish (tag-only).
+# PR: lint and unit tests only (file-filtered). Tag push: publish only (no lint/tests).
 on:
   pull_request:
     paths:
@@ -24,6 +24,7 @@ env:
 jobs:
   lint:
     name: Lint npmjs-publish
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-24.04
     steps:
       - name: Checkout Repository
@@ -44,6 +45,7 @@ jobs:
 
   test:
     name: Unit test npmjs-publish
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-24.04
     steps:
       - name: Checkout Repository

scripts/npmjs-publish/PLAN.md

@@ -29,7 +29,7 @@ todos:
 - **Tag:** `npmjs-publish@<semver>` for workflow and version parsing.
 - **Preflight:** rejects invalid semver and wrong tag prefixes; partial and full “already on registry” cases are **errors** (npm cannot overwrite an existing version).
 - **Build chain:** `build_packages.py` runs `make build-ts`, `shared-data lib-js`, `step-generation lib`, components `build-ts` + `lib`, protocol-visualization `build-ts` + `lib`, then optional manifest rewrite.
-- **Workflow** [`.github/workflows/npmjs-publish.yaml`](../../.github/workflows/npmjs-publish.yaml): PR path filters run lint + test; tag push runs publish preflight job with version from ref.
+- **Workflow** [`.github/workflows/npmjs-publish.yaml`](../../.github/workflows/npmjs-publish.yaml): PR path filters run lint + test; tag push runs **publish** preflight only (no lint or unit tests on the tag).
 - **step-generation:** npm-oriented `package.json` and `Makefile` targets (`build-ts`, `lib`, `pack`).
 
 **Not done yet:**

scripts/npmjs-publish/README.md

@@ -75,7 +75,7 @@ make build-packages-manifests-only VERSION=1.2.3
 Workflow: [`.github/workflows/npmjs-publish.yaml`](../../.github/workflows/npmjs-publish.yaml)
 
 - **pull_request** (paths: this workflow + `scripts/npmjs-publish/**`): **lint** and **unit test** jobs (parallel, no `needs`).
-- **push** tags `npmjs-publish@*`: same jobs plus **publish** job: resolve version from tag, run preflight (`publish.py`).
+- **push** tags `npmjs-publish@*`: **publish** job only: resolve version from tag, run preflight (`publish.py`). Lint and unit tests do not run on tag push.
 
 Tag pushes use the full ref as `--version` so `publish_core.resolve_version_input` accepts it.