@codyswann/lisa 3.33.5 → 3.33.7

package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-openclaw",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Connect staff roles to Telegram or Slack via OpenClaw — facilitator/specialist hub-and-spoke routing and repo-coding topics, for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-phaser/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-phaser",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Phaser 4 game-development rules for TypeScript projects",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-phaser/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-phaser",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Phaser 4 game-development rules for TypeScript projects",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-phaser-agy/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-phaser",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Phaser 4 game-development rules for TypeScript projects",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-phaser",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Phaser 4 game-development rules for TypeScript projects",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-phaser",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Phaser 4 game-development rules for TypeScript projects",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-rails/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-rails",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Ruby on Rails-specific hooks — RuboCop linting/formatting and ast-grep scanning on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-rails/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-rails",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Ruby on Rails-specific skills and hooks for RuboCop and ast-grep scanning on edit.",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-rails-agy/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-rails",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Ruby on Rails-specific hooks — RuboCop linting/formatting and ast-grep scanning on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-rails",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Ruby on Rails-specific hooks — RuboCop linting/formatting and ast-grep scanning on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-rails",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Ruby on Rails-specific hooks — RuboCop linting/formatting and ast-grep scanning on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-typescript/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-typescript",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "TypeScript-specific hooks — Prettier formatting, ESLint linting, ast-grep scanning, and error-suppression blocking on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-typescript/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-typescript",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "TypeScript-specific hooks for formatting, linting, and ast-grep scanning on edit.",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-typescript-agy/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-typescript",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "TypeScript-specific hooks — Prettier formatting, ESLint linting, ast-grep scanning, and error-suppression blocking on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-typescript",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "TypeScript-specific hooks — Prettier formatting, ESLint linting, ast-grep scanning, and error-suppression blocking on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-typescript",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "TypeScript-specific hooks — Prettier formatting, ESLint linting, ast-grep scanning, and error-suppression blocking on edit",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "Distributable LLM Wiki kernel — ingest, query, lint, and maintain a git-native markdown knowledge base across Claude and Codex.",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki-agy/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "3.33.5",
+  "version": "3.33.7",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/src/base/skills/lisa-git-submit-pr/SKILL.md

@@ -66,10 +66,17 @@ When updating an existing PR, preserve any existing linkage line unless the new
 After creating or updating the PR, always make the reverse link durable on the source work item when `work_item_ref` is available:
 
 1. Resolve the live PR URL with `gh pr view <pr-number> --json url --jq .url`.
-2. Invoke `lisa-tracker-sync` with the original work item ref, milestone `pr-ready`, `pr_url=<url>`, and `tracker_provider=<provider>` when known.
-3. The vendor sync skill must prefer the provider's native development-link primitive where one is available and verifiable.
-4. If native linkage is unavailable, unconfigured, or cannot be verified, the vendor sync skill must create or update a single managed `[lisa-pr-link]` comment on the work item containing the PR URL. The fallback comment is not optional; it is the ticket-side half of the two-way link.
-5. When the PR later merges, invoke `lisa-tracker-sync` again with milestone `pr-merged`, the same `pr_url`, and the merge SHA when available, so the managed backlink reflects the final state.
+2. Run the backlink command. It is the executable form of this requirement — it writes the managed `[lisa-pr-link]` comment on the work item, or updates the one already there, for every tracker Lisa supports:
+
+   ```bash
+   node scripts/lisa-work-item.mjs backlink --ref <work_item_ref> --pr-url <url>
+   ```
+
+   It is idempotent, so run it on every push rather than deciding whether it is needed. It refuses loudly for a tracker it cannot write, and never silently no-ops. Do not hand-post the comment, and do not describe the posting procedure anywhere: the same file that writes it is the file that checks it, which is what keeps producer and consumer from drifting.
+3. Invoke `lisa-tracker-sync` with the original work item ref, milestone `pr-ready`, `pr_url=<url>`, and `tracker_provider=<provider>` when known. That is the progress-note and status side; it is not what satisfies the traceability check.
+4. When the PR later merges, invoke `lisa-tracker-sync` again with milestone `pr-merged`, the same `pr_url`, and the merge SHA when available.
+
+**Why step 2 exists — do not "simplify" it away as redundant with the `Refs` line.** Under the non-closing rule above, GitHub never creates a native development link at all: that surface *is* the closing-reference mechanism, so no non-closing form can populate it. A PR carrying a perfectly correct `Refs` line still fails the required Work-Item Traceability check without the comment. The managed comment is the ticket-side half of the link, not a fallback for when native linkage happens to be absent.
 
 Do not report PR submission as fully synced while the PR body references the ticket but the ticket has neither a verified native PR link nor the managed backlink comment.
 

package/plugins/src/base/skills/lisa-github-sync/SKILL.md

@@ -57,8 +57,13 @@ Optional arguments include `pr_url=<url>` for the live pull request and `merge_s
 When `$ARGUMENTS` includes `pr_url=<url>` for `PR ready` or `PR merged`, ensure the GitHub Issue has a durable ticket -> PR link:
 
 1. Make sure the PR body contains `Refs #<n>` (or the fully qualified cross-repo form) — never a closing keyword, per the GitHub rule in `lisa-git-submit-pr`. Read the issue side with `gh api graphql` against `issue.timelineItems`, or `gh issue view <number> --json closedByPullRequestsReferences`. **Not** `gh issue view --json timelineItems`: `timelineItems` is not a supported field for that command, so the check silently returns nothing useful rather than failing loudly.
-2. Post or update a single managed issue comment starting with `[lisa-pr-link]`. Include the PR URL, milestone (`pr-ready` or `pr-merged`), and merge SHA when available. This is **unconditional**, not contingent on step 1 failing: under the non-closing rule GitHub never creates a native development link (that surface is the closing-reference mechanism), so this comment is the only ticket-side backlink there will be.
-3. Keep the fallback idempotent: search existing comments for `[lisa-pr-link]` and the PR URL; update/replace that managed comment where the provider allows updates, otherwise skip when the current body already matches. Do not append duplicate backlink comments on reruns.
+2. Establish the managed backlink comment by running the command that owns it — never by hand, and never by describing the procedure here:
+
+   ```bash
+   node scripts/lisa-work-item.mjs backlink --ref <work-item> --pr-url <url>
+   ```
+
+   It creates the `[lisa-pr-link]` comment or updates the one already present, instead of appending duplicates, so it is safe to run on every milestone. This is **unconditional**, not contingent on step 1 failing: under the non-closing rule GitHub never creates a native development link (that surface is the closing-reference mechanism), so this comment is the only ticket-side backlink there will be — run it whether or not native linkage exists or cannot be verified, because the required Work-Item Traceability check reads this comment and nothing else guarantees one. The comment carries the marker and the PR URL only; the milestone (`pr-ready` / `pr-merged`) and merge SHA belong in the milestone progress note, so that a rerun at a new milestone still converges on one backlink comment.
 
 Native GitHub linkage cannot be verified under the non-closing rule because it is never created in the first place, so the managed comment is not a contingency here — it is the mechanism. The issue must show the PR from at least one ticket-side surface, and this is the only one available.
 

package/plugins/src/base/skills/lisa-implement/SKILL.md

@@ -341,7 +341,7 @@ Before shutting down the team, execute the Verify flow:
 5. Commit ALL outstanding changes in logical batches on the branch (minus sensitive data/information) — not just changes made by the agent team. This includes pre-existing uncommitted changes that were on the branch before the plan started. Do NOT filter commits to only "task-related" files. If it shows up in git status, it gets committed (unless it contains secrets).
 6. Push the changes - if any pre-push hook blocks you, create a task for the agent team to fix the error/problem whether it was pre-existing or not
 7. Open a pull request with auto-merge on via `lisa-git-submit-pr`, targeting the **base branch resolved from the ticket's environment** (`target_branch=<base>`, per the branch step above), and including the mandatory work-item ref so the PR can be linked natively to the source issue.
-7a. Confirm two-way linkage before treating PR submission as complete: the PR body/title/branch must reference the work item, and the work item must have either a verified native PR link or a single managed `[lisa-pr-link]` fallback comment from `lisa-tracker-sync`.
+7a. Confirm two-way linkage before treating PR submission as complete: the PR body/title/branch must reference the work item, and the work item must have either a verified native PR link or the single managed `[lisa-pr-link]` comment. Establish that comment with `node scripts/lisa-work-item.mjs backlink --ref <work_item_ref> --pr-url <url>` rather than posting it by hand — it is idempotent, and it is the same file that enforces the check, so producer and consumer cannot drift.
 8. PR Watch Loop: Drive the PR to merge via the `drive-pr-to-merge` skill — the single source of truth for clearing every blocker (auto-merge with direct-merge fallback, `BEHIND` re-sync, conflict resolution, failing-check fixes, human + bot review-comment handling with thread resolution, stale `CHANGES_REQUESTED` dismissal, and post-merge ancestry verification). `git-submit-pr` already invokes it; if you reach this step with a PR already open, invoke `drive-pr-to-merge` directly with the PR number. For a large review backlog you may fan the code-fix work out to the agent team, but `drive-pr-to-merge` owns the loop and the terminal conditions — do not re-implement them.
 9. Merge the PR, then refresh the ticket-side backlink with `lisa-tracker-sync <work_item_ref> pr-merged pr_url=<url> merge_sha=<sha> tracker_provider=<provider>`.
 10. Monitor the deploy action that triggers automatically from the successful merge

package/plugins/src/base/skills/lisa-jira-sync/SKILL.md

@@ -55,8 +55,13 @@ Before adding a comment, check for an existing milestone comment to avoid duplic
 When `$ARGUMENTS` includes `pr_url=<url>` for `PR ready` or `PR merged`, ensure the JIRA ticket has a durable ticket -> PR link:
 
 1. Prefer the JIRA development-link surface when the site's GitHub/JIRA integration or remote-link API is available through `lisa-atlassian-access`; verify by re-reading the ticket's remote links / development metadata.
-2. If native linkage is unavailable, unconfigured, cross-system, or cannot be verified, create or update a single managed JIRA comment containing the PR URL. The comment must start with `[lisa-pr-link]` and include the milestone (`pr-ready` or `pr-merged`) and merge SHA when available.
-3. Keep the fallback idempotent: read existing comments, find the `[lisa-pr-link]` comment for the same PR URL, and update/skip it instead of appending duplicates. If the current access layer cannot update comments in place, skip when an identical managed comment already exists and otherwise add exactly one replacement comment with the stable marker.
+2. Establish the managed backlink comment by running the command that owns it — never by hand, and never by describing the procedure here:
+
+   ```bash
+   node scripts/lisa-work-item.mjs backlink --ref <work-item> --pr-url <url>
+   ```
+
+   It creates the `[lisa-pr-link]` comment or updates the one already present, instead of appending duplicates, so it is safe to run on every milestone. This is **unconditional** — run it whether or not native linkage exists or cannot be verified, because the required Work-Item Traceability check reads this comment and nothing else guarantees one. The comment carries the marker and the PR URL only; the milestone (`pr-ready` / `pr-merged`) and merge SHA belong in the milestone progress note, so that a rerun at a new milestone still converges on one backlink comment.
 
 The PR body/branch issue key is the PR -> ticket side. This step is the required ticket -> PR side.
 

package/plugins/src/base/skills/lisa-linear-sync/SKILL.md

@@ -72,8 +72,13 @@ Call `lisa-linear-access operation: save-comment({issueId: <id>, body: <comment>
 When `$ARGUMENTS` includes `pr_url=<url>` for `pr-ready` or `pr-merged`, ensure the Linear Issue has a durable ticket -> PR link:
 
 1. Prefer Linear's native GitHub attachment / pull request link when the integration has attached the PR through the branch name, PR title, or PR body issue identifier. Verify by re-reading the Issue and its attachments / relations where the Linear access layer exposes them.
-2. If native linkage is unavailable, unconfigured, cross-system, or cannot be verified, create or update a single managed Linear comment containing the PR URL. The comment must start with `[lisa-pr-link]` and include the milestone (`pr-ready` or `pr-merged`) and merge SHA when available.
-3. Keep the fallback idempotent: read existing comments where the access layer exposes them, find the `[lisa-pr-link]` comment for the same PR URL, and update/skip it instead of appending duplicates. If comment update is unavailable, skip when an identical managed comment already exists and otherwise add exactly one replacement comment with the stable marker.
+2. Establish the managed backlink comment by running the command that owns it — never by hand, and never by describing the procedure here:
+
+   ```bash
+   node scripts/lisa-work-item.mjs backlink --ref <work-item> --pr-url <url>
+   ```
+
+   It creates the `[lisa-pr-link]` comment or updates the one already present, instead of appending duplicates, so it is safe to run on every milestone. This is **unconditional** — run it whether or not native linkage exists or cannot be verified, because the required Work-Item Traceability check reads this comment and nothing else guarantees one. The comment carries the marker and the PR URL only; the milestone (`pr-ready` / `pr-merged`) and merge SHA belong in the milestone progress note, so that a rerun at a new milestone still converges on one backlink comment.
 
 The PR branch/title/body identifier is the PR -> Linear side. This phase is the required Linear -> PR side.
 

package/plugins/src/base/skills/lisa-tracker-sync/SKILL.md

@@ -52,8 +52,13 @@ When `$ARGUMENTS` includes `pr_url=<url>` with milestone `pr-ready` or `pr-merge
 
 1. Prefer the provider's native development-link primitive when Lisa can write and verify it for that provider.
 2. Verify the native link using the provider read surface when available.
-3. If the native link is unavailable, unconfigured, cross-system, or cannot be verified, create or update one managed backlink comment on the work item containing the PR URL and current milestone.
-4. Keep the comment idempotent by using a stable marker such as `[lisa-pr-link]`; reruns update or skip the existing managed comment rather than appending duplicates.
+3. Whether or not the native link exists or cannot be verified, establish the managed backlink comment with the one command that owns it:
+
+   ```bash
+   node scripts/lisa-work-item.mjs backlink --ref <work-item> --pr-url <url>
+   ```
+
+4. That command is idempotent by construction — it updates the existing `[lisa-pr-link]` comment rather than appending duplicates — and it refuses loudly for a tracker it cannot write. Do not restate its procedure in a vendor skill; the file that writes the comment is the file that checks it, and that is what stops the two from drifting.
 
 This is the reverse half of `lisa-git-submit-pr`'s PR body linkage. A PR that mentions a ticket is not considered fully synced until the ticket also has either a verified native PR link or the managed fallback comment.
 

package/scripts/check-conflict-markers.mjs

@@ -299,7 +299,7 @@ export function main(argv, io = {}) {
     scanned: files.length,
   });
   out.write(
-    (opts.json ? JSON.stringify(report, null, 2) : humanReport(report)) + "\n"
+    `${opts.json ? JSON.stringify(report, null, 2) : humanReport(report)}\n`
   );
   return results.length === 0 ? 0 : 1;
 }

package/scripts/check-required-check-promotions.mjs

@@ -140,6 +140,10 @@ import path from "node:path";
 import { fileURLToPath } from "node:url";
 import yaml from "js-yaml";
 
+// Literals named once — each was repeated enough times that a typo in one
+// copy would diverge silently.
+const HEADROOM_EVIDENCE_MISSING = "headroom-evidence-missing";
+
 /** Integration id GitHub Actions reports status checks under. */
 export const ACTIONS_INTEGRATION_ID = 15_368;
 
@@ -343,7 +347,7 @@ function budgetProblems(budget) {
   if (typeof subject !== "string" || typeof measuredOn !== "string") {
     return [
       {
-        rule: "headroom-evidence-missing",
+        rule: HEADROOM_EVIDENCE_MISSING,
         detail:
           "every headroom.budgets[] entry needs a subject and a measured_on_subject",
       },
@@ -391,7 +395,7 @@ function provenanceProblems(observedOn) {
   }
   return [
     {
-      rule: "headroom-evidence-missing",
+      rule: HEADROOM_EVIDENCE_MISSING,
       detail:
         'headroom.observed_on must be "pass": the run that produced observed_worst_ms must have COMPLETED within its budget, because a duration reported alongside a timeout measures contention, not cost',
     },
@@ -414,7 +418,7 @@ function ratioProblems(budgetMs, observedMs) {
   ) {
     return [
       {
-        rule: "headroom-evidence-missing",
+        rule: HEADROOM_EVIDENCE_MISSING,
         detail:
           "headroom needs a positive budget_ms and a positive observed_worst_ms",
       },
@@ -481,7 +485,7 @@ export function headroomProblems(headroom) {
   for (const [field, detail] of prose) {
     const value = headroom[field];
     if (typeof value !== "string" || value.trim() === "") {
-      problems.push({ rule: "headroom-evidence-missing", detail });
+      problems.push({ rule: HEADROOM_EVIDENCE_MISSING, detail });
     }
   }
   if (problems.length > 0) return problems;

package/scripts/detect-stale-workflow-inputs.mjs

@@ -231,9 +231,7 @@ function humanTable(report) {
  * @returns {void}
  */
 function emitReport(out, report, json) {
-  out.write(
-    (json ? JSON.stringify(report, null, 2) : humanTable(report)) + "\n"
-  );
+  out.write(`${json ? JSON.stringify(report, null, 2) : humanTable(report)}\n`);
 }
 
 /**

package/scripts/fix-namespace-test-assertions.mjs

@@ -2,7 +2,7 @@
 /**
  * Fix test assertions and a few skill references after lisa-* skill / lisa: command namespace migration.
  */
-import { readFileSync, writeFileSync, readdirSync, statSync } from "node:fs";
+import { readFileSync, writeFileSync } from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 

package/scripts/generate-agy-plugin-artifacts.mjs

@@ -44,6 +44,11 @@ import { fileURLToPath } from "node:url";
 import { nestCommandsUnderLisa } from "./lib/nest-plugin-commands.mjs";
 import { invokedAsScript } from "./lib/invoked-as-script.mjs";
 
+// Literals named once — each was repeated enough times that a typo in one
+// copy would diverge silently.
+const CLAUDE_PLUGIN_DIR = ".claude-plugin";
+const PLUGIN_MANIFEST = "plugin.json";
+
 const REPO_ROOT = path.resolve(
   path.dirname(fileURLToPath(import.meta.url)),
   ".."
@@ -115,7 +120,7 @@ function copyDir(src, dst, keep = () => true) {
  * @param {string} version Version string for the manifest.
  */
 export function generateAgyVariant(srcDir, outDir, version) {
-  const claudeManifest = path.join(srcDir, ".claude-plugin", "plugin.json");
+  const claudeManifest = path.join(srcDir, CLAUDE_PLUGIN_DIR, PLUGIN_MANIFEST);
   if (!fs.existsSync(claudeManifest)) return;
 
   const policy = readPolicy();
@@ -148,8 +153,8 @@ export function generateAgyVariant(srcDir, outDir, version) {
     }
     // Skip the .claude-plugin manifest — we'll write a bare plugin.json instead.
     if (
-      relPath === path.join(".claude-plugin", "plugin.json") ||
-      relPath === ".claude-plugin"
+      relPath === path.join(CLAUDE_PLUGIN_DIR, PLUGIN_MANIFEST) ||
+      relPath === CLAUDE_PLUGIN_DIR
     ) {
       return false;
     }
@@ -181,11 +186,11 @@ export function generateAgyVariant(srcDir, outDir, version) {
   delete manifest.hooks;
   delete manifest.mcpServers;
 
-  const bareManifestPath = path.join(outDir, "plugin.json");
-  fs.writeFileSync(bareManifestPath, JSON.stringify(manifest, null, 2) + "\n");
+  const bareManifestPath = path.join(outDir, PLUGIN_MANIFEST);
+  fs.writeFileSync(bareManifestPath, `${JSON.stringify(manifest, null, 2)}\n`);
 
   // 3. Ensure no .claude-plugin/ directory survives.
-  const ghostDir = path.join(outDir, ".claude-plugin");
+  const ghostDir = path.join(outDir, CLAUDE_PLUGIN_DIR);
   if (fs.existsSync(ghostDir)) {
     fs.rmSync(ghostDir, { recursive: true, force: true });
   }
@@ -328,7 +333,7 @@ function emitAgyPluginHooks(srcDir, outDir, sourceHooks, installDirName) {
   );
   fs.writeFileSync(
     path.join(outDir, "hooks.json"),
-    JSON.stringify(hooksConfig, null, 2) + "\n"
+    `${JSON.stringify(hooksConfig, null, 2)}\n`
   );
 
   // Copy the agy-protocol scripts into the variant's hooks/ subdir.

package/scripts/generate-codex-plugin-artifacts.mjs

@@ -38,6 +38,11 @@ import fs from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 
+// Literals named once — each was repeated enough times that a typo in one
+// copy would diverge silently.
+const CODEX_PLUGIN_DIR = ".codex-plugin";
+const HOOKS_MANIFEST = "hooks.json";
+
 const REPO_ROOT = path.resolve(
   path.dirname(fileURLToPath(import.meta.url)),
   ".."
@@ -469,7 +474,7 @@ function writeSkillAgentsInRoot(skillsDir) {
 function codexSkillRoots(pluginDir) {
   return [
     path.join(pluginDir, "skills"),
-    path.join(pluginDir, ".codex-plugin", "skills"),
+    path.join(pluginDir, CODEX_PLUGIN_DIR, "skills"),
   ].filter(skillRoot => fs.existsSync(skillRoot));
 }
 
@@ -482,7 +487,7 @@ function codexSkillRoots(pluginDir) {
  * @returns {readonly string[]} Generated skill names.
  */
 export function emitCommandSkills(pluginDir) {
-  const generatedRoot = path.join(pluginDir, ".codex-plugin", "skills");
+  const generatedRoot = path.join(pluginDir, CODEX_PLUGIN_DIR, "skills");
   const commandsRoot = path.join(pluginDir, "commands");
   if (!fs.existsSync(commandsRoot)) {
     return [];
@@ -536,7 +541,7 @@ export function emitCommandSkills(pluginDir) {
  */
 export function emitCodexSkillVariants(pluginDir) {
   const authoredRoot = path.join(pluginDir, "skills");
-  const codexRoot = path.join(pluginDir, ".codex-plugin", "skills");
+  const codexRoot = path.join(pluginDir, CODEX_PLUGIN_DIR, "skills");
   fs.rmSync(codexRoot, { force: true, recursive: true });
   if (!fs.existsSync(authoredRoot)) return [];
 
@@ -799,14 +804,14 @@ function main() {
  * @param {object} claudeManifest Parsed contents of .claude-plugin/plugin.json.
  */
 export function emitCodexHooks(pluginDir, claudeManifest) {
-  const codexPluginDir = path.join(pluginDir, ".codex-plugin");
-  const hooksJsonPath = path.join(codexPluginDir, "hooks.json");
+  const codexPluginDir = path.join(pluginDir, CODEX_PLUGIN_DIR);
+  const hooksJsonPath = path.join(codexPluginDir, HOOKS_MANIFEST);
   // Purge the older layouts so a rebuilt plugin never ships a stale/duplicate
   // file:
   //   - <plugin-root>/hooks/hooks.json     (2.121–2.124; broke Claude startup)
   //   - <plugin-root>/.codex-plugin/hooks/ (pre-2.121 copied-scripts dir)
   // The shared .sh scripts at <plugin-root>/hooks/ are left untouched.
-  fs.rmSync(path.join(pluginDir, "hooks", "hooks.json"), { force: true });
+  fs.rmSync(path.join(pluginDir, "hooks", HOOKS_MANIFEST), { force: true });
   fs.rmSync(path.join(codexPluginDir, "hooks"), {
     force: true,
     recursive: true,
@@ -861,7 +866,7 @@ function writeCodexManifest(pluginDir, claudeManifest, pluginName, version) {
     },
   };
 
-  const manifestDir = path.join(pluginDir, ".codex-plugin");
+  const manifestDir = path.join(pluginDir, CODEX_PLUGIN_DIR);
   fs.mkdirSync(manifestDir, { recursive: true });
   fs.writeFileSync(
     path.join(manifestDir, "plugin.json"),
@@ -871,7 +876,7 @@ function writeCodexManifest(pluginDir, claudeManifest, pluginName, version) {
 
 export function componentPointers(pluginDir) {
   return {
-    ...(fs.existsSync(path.join(pluginDir, ".codex-plugin", "skills"))
+    ...(fs.existsSync(path.join(pluginDir, CODEX_PLUGIN_DIR, "skills"))
       ? { skills: "./.codex-plugin/skills/" }
       : fs.existsSync(path.join(pluginDir, "skills"))
         ? { skills: "./skills/" }
@@ -879,7 +884,7 @@ export function componentPointers(pluginDir) {
     ...(fs.existsSync(path.join(pluginDir, ".mcp.json"))
       ? { mcpServers: "./.mcp.json" }
       : {}),
-    ...(fs.existsSync(path.join(pluginDir, ".codex-plugin", "hooks.json"))
+    ...(fs.existsSync(path.join(pluginDir, CODEX_PLUGIN_DIR, HOOKS_MANIFEST))
       ? { hooks: "./.codex-plugin/hooks.json" }
       : {}),
   };

package/scripts/generate-copilot-plugin-artifacts.mjs

@@ -247,7 +247,7 @@ export function generateCopilotVariant(srcDir, outDir, version) {
     }
   }
 
-  fs.writeFileSync(manifestPath, JSON.stringify(manifest, null, 2) + "\n");
+  fs.writeFileSync(manifestPath, `${JSON.stringify(manifest, null, 2)}\n`);
 
   // 4. Filter the hooks/ directory.
   const hooksDir = path.join(outDir, "hooks");

package/scripts/generate-cursor-plugin-artifacts.mjs

@@ -336,7 +336,7 @@ export function generateCursorVariant(srcDir, outDir, version) {
   manifest.version = version;
   const cursorHooks = buildCursorHooksJson(manifest.hooks ?? {});
   delete manifest.hooks;
-  fs.writeFileSync(manifestPath, JSON.stringify(manifest, null, 2) + "\n");
+  fs.writeFileSync(manifestPath, `${JSON.stringify(manifest, null, 2)}\n`);
 
   // 2a. Emit hooks/hooks.json in Cursor's native shape when any hooks survive.
   if (cursorHooks) {
@@ -344,7 +344,7 @@ export function generateCursorVariant(srcDir, outDir, version) {
     fs.mkdirSync(cursorHooksDir, { recursive: true });
     fs.writeFileSync(
       path.join(cursorHooksDir, "hooks.json"),
-      JSON.stringify(cursorHooks, null, 2) + "\n"
+      `${JSON.stringify(cursorHooks, null, 2)}\n`
     );
   }
 

package/scripts/generate-upstream-evidence-manifest.mjs

@@ -123,7 +123,7 @@ const existingOutput = existsSync(outputPath)
   : "";
 const existingPublicCommits = [
   ...existingOutput.matchAll(
-    /^\s+(?:"([0-9][a-f0-9]{39})"|([a-f][a-f0-9]{39})):\s+true,$/gmu
+    /^\s+(?:"(\d[a-f0-9]{39})"|([a-f][a-f0-9]{39})):\s+true,$/gmu
   ),
 ]
   .map(match => match[1] ?? match[2] ?? "")

package/scripts/lisa-assert-eas-profile.mjs

@@ -88,9 +88,9 @@ export function resolveProfile(profiles, name) {
     if (!profile) {
       const known = Object.keys(profiles ?? {});
       throw new Error(
-        `eas.json has no build profile called "${current}"` +
-          (current === name ? "" : ` (reached by extending from "${name}")`) +
-          `. It defines: ${known.length ? known.join(", ") : "none"}.`
+        `eas.json has no build profile called "${current}"${
+          current === name ? "" : ` (reached by extending from "${name}")`
+        }. It defines: ${known.length ? known.join(", ") : "none"}.`
       );
     }
     seen.add(current);