@bushizhanghan/dsh-publish-gate 0.13.0 → 0.13.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.13.1
+
+- The tag check now asks `git ls-remote origin` instead of `git tag --points-at HEAD`. When the commit is pushed
+  through the GitHub API the local HEAD is not the remote one, so the local form reported `git-tag-missing` on
+  seven of my nine plugins whose tag existed all along. It now compares the tag against the shipped branch
+  head and stays silent when the remote cannot be reached — a check that cannot see the truth must not claim
+  a defect.
 ## 0.13.0
 
 - `market-fork-of-listed` / `market-fork-undisclosed` (warn): the repo is a GitHub fork whose parent is

package/README.md

@@ -223,7 +223,7 @@ dsh plugin --profile desktop add @bushizhanghan/dsh-publish-gate
 ## 开发
 
 ```sh
-node --test                      # 107 个引擎单测
+node --test                      # 108 个引擎单测
 node test/integration.test.mjs   # 27 条集成断言（9 个真实插件作 fixture）
 ```
 

package/lib/engine.js

@@ -2303,11 +2303,43 @@ export async function checkTraceability(dir, opts = {}) {
     }
   }
 
-  // 2. No tag on HEAD: nothing points at these bytes afterwards.
+  // 2. A tag on the bytes that actually shipped. When the commit was pushed
+  // through the GitHub API the local HEAD is not the remote one, so a local
+  // `git tag --points-at HEAD` reports a missing tag that exists — ask the
+  // remote instead, and say nothing when it cannot be reached.
+  const remote = opts.remote === undefined ? await readRemoteRefs(dir) : opts.remote
   const atHead = opts.atHead === undefined ? await gitOut(dir, ['tag', '--points-at', 'HEAD']) : opts.atHead
-  const tags = typeof atHead === 'string' ? atHead.split('\n').map((t) => t.trim()).filter(Boolean) : []
+  const localTags = typeof atHead === 'string' ? atHead.split('\n').map((t) => t.trim()).filter(Boolean) : []
   const wanted = [`v${version}`, version]
-  if (typeof atHead === 'string' && !wanted.some((t) => tags.includes(t))) {
+  const tagOnRemote = remote ? remote.tags[`v${version}`] || remote.tags[version] || null : null
+  const tagIsOnShipped = Boolean(remote && tagOnRemote && tagOnRemote === remote.head)
+  const localHasIt = wanted.some((t) => localTags.includes(t))
+
+  if (remote) {
+    // The remote is the truth: the tag either points at the shipped commit or not.
+    if (tagIsOnShipped) return out
+    const exists = Boolean(tagOnRemote) || (opts.tagExists !== undefined ? Boolean(opts.tagExists) : Boolean(await gitOut(dir, ['rev-parse', '--verify', '--quiet', `refs/tags/v${version}`])))
+    out.push(
+      exists
+        ? F(
+            'git-tag-elsewhere',
+            'warn',
+            `tag v${version} does not point at the shipped commit`,
+            `The tag is on ${String(tagOnRemote || 'another commit').slice(0, 12)} but the branch head is ${String(remote.head).slice(0, 12)}. A release built from that tag is not what you just published.`,
+            `Move it: git tag -f v${version} <shipped-sha> && git push -f origin v${version}`,
+          )
+        : F(
+            'git-tag-missing',
+            'warn',
+            `no git tag v${version} on the shipped commit`,
+            'After publishing, the only record of these bytes is the registry. A GitHub release, a changelog entry or a market listing cannot point at a commit that was never tagged.',
+            `git tag v${version} && git push origin v${version}`,
+          ),
+    )
+    return out
+  }
+
+  if (typeof atHead === 'string' && !localHasIt) {
     const exists = opts.tagExists === undefined ? await gitOut(dir, ['rev-parse', '--verify', '--quiet', `refs/tags/v${version}`]) : opts.tagExists ? 'sha' : null
     out.push(
       exists
@@ -2330,6 +2362,22 @@ export async function checkTraceability(dir, opts = {}) {
   return out
 }
 
+/** `{ head, tags }` from one `git ls-remote`, or null when it cannot be read. */
+async function readRemoteRefs(dir) {
+  const raw = await gitOut(dir, ['ls-remote', '--heads', '--tags', 'origin'])
+  if (typeof raw !== 'string' || raw.length === 0) return null
+  const tags = {}
+  let head = null
+  for (const line of raw.split('\n')) {
+    const [sha, ref] = line.split('\t')
+    if (!sha || !ref) continue
+    if (ref === 'refs/heads/main' || ref === 'refs/heads/master') head = sha
+    const m = /^refs\/tags\/(.+?)(?:\^\{\})?$/.exec(ref)
+    if (m) tags[m[1]] = sha // a dereferenced annotated tag overwrites the tag object, which is what we want
+  }
+  return head ? { head, tags } : null
+}
+
 // =====================================================================
 // Orchestration
 // =====================================================================

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bushizhanghan/dsh-publish-gate",
-  "version": "0.13.0",
+  "version": "0.13.1",
   "description": "Pre-publish gate for DSH plugins: manifest, dependencies, artifact, runtime, browser half, traceability and market readiness — plus an input fuzzer that drives every declared parameter with the wrong type. 发布前的七层门禁 + 入参模糊测试。",
   "type": "module",
   "main": "lib/index.js",