From 39b6fcb2733873eead38b7e58087a0f211bdf831 Mon Sep 17 00:00:00 2001
From: Kohei Otsuka <kochizufan@gmail.com>
Date: Fri, 30 Oct 2026 13:04:14 +0900
Subject: [PATCH] test: share the public wording check expression with the
 sibling packages

The check now uses the same expression as Teiresias and Daedalus/Atlas: it
also flags release-branch names, review row labels, working-group names and
private repository names, and leaves section references of external standards
(RFC, ISO, OGC) and ordinary uses of the word for design alone. Samples for
both directions are tested.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 tests/public-surface-wording.test.ts | 76 +++++++++++++++++++++++++++-
 1 file changed, 74 insertions(+), 2 deletions(-)

diff --git a/tests/public-surface-wording.test.ts b/tests/public-surface-wording.test.ts
index 3c870d1..fab6951 100644
--- a/tests/public-surface-wording.test.ts
+++ b/tests/public-surface-wording.test.ts
@@ -5,13 +5,73 @@ import { describe, expect, it } from "vitest";
 
 // Files that reach readers (the npm tarball, the README files and the changelog) must not
 // point at the private development notes this package was built from: links into the
-// private repository, internal task and milestone labels, or section numbers of
+// private repository, internal task, milestone and review labels, month-level stage names,
+// the row numbers of the review table, design decision numbers, or section numbers of
 // unpublished documents. dist is generated from src, so src is what is checked.
 
 const packageRoot = fileURLToPath(new URL("..", import.meta.url));
 
+/**
+ * Internal references in the wording of the published files.
+ *
+ * Each alternative is kept narrow on purpose, so ordinary code and text (EPSG codes,
+ * `UTJCP:ZONE-B`, `metresPerUnit`, `UTF-8`, `ISO-8601`, `x-skew`) stay clean. The samples
+ * used by the tests below are the guard that the expression is neither too narrow nor too
+ * wide.
+ */
 const INTERNAL_REFERENCE =
-  /docs\/superpowers|\bm[0-9]+[ab]?-t[0-9]+|Cycle 1|post-cycle|仮仕様|AC-m[0-9]|HR-m[0-9]|SR-m[0-9]|maplat-v4-crs-libs|provisional specification|§[0-9]/;
+  /docs\/superpowers|\bPR-T[0-9]+[A-Za-z]?|\btrain\/[0-9]{4}-[0-9]{2}|\b(?:MAJ|MIN)-[0-9]+|\b(?:Minor|Info)-[0-9]+|トレイン|照合表|第\s*[0-9]+\s*便|サイクル\s*[0-9]+|NayutaProducts|\bDSH\b|\bm[0-9]+[ab]?-t[0-9]+|\bCycle ?[0-9]|post-cycle|仮仕様|AC-m[0-9]|HR-m[0-9]|SR-m[0-9]|maplat-v4-crs-libs|provisional specification|(?<!\b(?:RFC|ISO|OGC)\s?[0-9]{1,5}\s*)§[0-9]|\b(?:dec|nov|oct)2[0-9]|\b[A-Z]-[0-9]+\b|\bDM-[A-Z]?[0-9]|\bK[0-9]+\b|設計書|rule-00|那由多|INF-[0-9]|\bAC-?[0-9]/;
+
+/**
+ * Lines the expression must flag. The labels are made up (they follow the shapes of the
+ * real ones without naming any). The expression and the samples live only in this file;
+ * tests/ is not part of the published tarball.
+ */
+const MUST_MATCH = [
+  "dec29",
+  "D-9",
+  "T-99",
+  "DM-Z9",
+  "設計書",
+  "m9-t9",
+  "§9.9",
+  "K99",
+  "PR-T9z",
+  "train/2099-01",
+  "MAJ-9",
+  "MIN-9",
+  "Minor-9",
+  "Info-9",
+  "トレイン",
+  "照合表",
+  "第 9 便",
+  "サイクル 9",
+  "NayutaProducts",
+  "DSH",
+  "// dec29 probe",
+  "// see 設計書 D-9 probe",
+  "// T-99 probe",
+  "// DM-Z9 probe",
+];
+
+/** Ordinary code and text the expression must leave alone. */
+const MUST_NOT_MATCH = [
+  "EPSG:6674",
+  "UTJCP:ZONE-B",
+  "metresPerUnit",
+  "xScale",
+  "DEFAULT_TIMEOUT_MS",
+  "adapter-timeout",
+  "UTF-8",
+  "ISO-8601",
+  "x-skew",
+  "failure.reason: 'timeout'",
+  "ok_with_warnings",
+  "RFC 7946 §3.1",
+  "ISO 19111 §7.2",
+  "設計上の理由で",
+  "設計された",
+];
 
 function walk(path: string): string[] {
   if (statSync(path).isDirectory()) {
@@ -40,6 +100,18 @@ describe("public files carry no internal references", () => {
     expect(files.length).toBeGreaterThan(0);
   });
 
+  it("flags the internal references reviewers reported", () => {
+    for (const sample of MUST_MATCH) {
+      expect(INTERNAL_REFERENCE.test(sample), sample).toBe(true);
+    }
+  });
+
+  it("leaves ordinary code and text alone", () => {
+    for (const sample of MUST_NOT_MATCH) {
+      expect(INTERNAL_REFERENCE.test(sample), sample).toBe(false);
+    }
+  });
+
   it("has no internal reference in the published sources, README files and changelog", () => {
     const hits: string[] = [];
     for (const file of files) {