@phuc1403/musketeer 0.5.0 → 0.7.0

package/template/.claude/hooks/sync-adr-toc.cjs

@@ -1,111 +1,146 @@
-#!/usr/bin/env node
-// PostToolUse hook: after any `adr` command, rebuild docs/adr/README.md from the
-// ADR files themselves — one row per ADR with its title and current status.
-//
-// The index is fully derived, so it can never drift: every run re-reads the
-// source. That is why there is no need to be clever about which adr subcommand
-// ran, or to protect the file from being rewritten.
-//
-// Also runnable directly:  node sync-adr-toc.cjs --write
-//
-// Fails open at exit 0 on any problem — a hook must never block a session.
-
-const fs = require('fs');
-const path = require('path');
-
-const root = process.env.CLAUDE_PROJECT_DIR || process.cwd();
-const ADR_FILE = /^\d+-.*\.md$/;
-
-// adr-tools reads its directory from a committed `.adr-dir` (default `doc/adr`).
-function adrDir() {
-  try {
-    return fs.readFileSync(path.join(root, '.adr-dir'), 'utf8').trim() || 'doc/adr';
-  } catch {
-    return 'doc/adr';
-  }
-}
-
-// Was this Bash call an adr command? (`VISUAL=true EDITOR=true adr new …` counts;
-// the internal `adr-new` script name does not, hence the required whitespace.)
-function isAdrCommand() {
-  try {
-    const payload = JSON.parse(fs.readFileSync(0, 'utf8'));
-    return /\badr\s/.test(payload?.tool_input?.command || '');
-  } catch {
-    return false;
-  }
-}
-
-// First line, minus the leading "# " and adr-tools' own number prefix — it writes
-// "# 12: Use X over Y" for 0012-….md. Dropping the unpadded 12 keeps the padded
-// stem in the ADR column the only number in the file, so nothing here can be
-// mistaken for the stem `adr new -s` expects.
-function title(body, fallback) {
-  const first = body.split(/\r?\n/)[0] || '';
-  return first.replace(/^#\s*(\d+\s*[:.]\s*)?/, '').trim() || fallback;
-}
-
-// First non-empty line of the `## Status` section. adr-tools maintains this:
-// "Accepted", or "Superceded by [4: …](0004-….md)" after `adr new -s`.
-function status(body) {
-  const lines = body.split(/\r?\n/);
-  const start = lines.findIndex((l) => /^##\s+Status\s*$/i.test(l));
-  if (start === -1) return 'Unknown';
-  for (const line of lines.slice(start + 1)) {
-    if (/^##\s/.test(line)) break;
-    if (line.trim()) return line.trim();
-  }
-  return 'Unknown';
-}
-
-function render(dir, files) {
-  const rows = files.map((name) => {
-    const body = fs.readFileSync(path.join(root, dir, name), 'utf8');
-    const stem = name.replace(/\.md$/, ''); // padded — what `adr new -s` needs
-    const cell = (s) => s.replace(/\|/g, '\\|'); // never break the table
-    return `| [${stem}](./${name}) | ${cell(title(body, stem))} | ${cell(status(body))} |`;
-  });
-
-  return [
-    '# Architecture Decision Records',
-    '',
-    'Generated from the ADR files by the `sync-adr-toc` hook — do not hand-edit.',
-    '',
-    '| ADR | Title | Status |',
-    '|-----|-------|--------|',
-    ...rows,
-    '',
-  ].join('\n');
-}
-
-function main() {
-  if (!process.argv.includes('--write') && !isAdrCommand()) return;
-
-  const dir = adrDir();
-  const files = fs
-    .readdirSync(path.join(root, dir))
-    .filter((f) => ADR_FILE.test(f))
-    .sort();
-  if (files.length === 0) return;
-
-  const target = path.join(root, dir, 'README.md');
-  const next = render(dir, files);
-
-  let current = null;
-  try {
-    current = fs.readFileSync(target, 'utf8');
-  } catch {
-    /* no index yet */
-  }
-  if (current === next) return; // already in sync
-
-  fs.writeFileSync(target, next);
-  process.stdout.write(`Regenerated ${dir}/README.md (${files.length} ADRs).\n`);
-}
-
-try {
-  main();
-} catch {
-  /* fail open */
-}
-process.exit(0);
+#!/usr/bin/env node
+// PostToolUse hook: after any `adr` command, rebuild docs/adr/README.md from the
+// ADR files themselves — one row per ADR with its title and current status.
+//
+// The index is fully derived, so it can never drift: every run re-reads the
+// source. That is why there is no need to be clever about which adr subcommand
+// ran, or to protect the file from being rewritten.
+//
+// Also deletes the tool's own `decisions.md`. Every `adr new` regenerates that
+// file unconditionally (not just on `adr generate toc`), which would leave two
+// competing indexes side by side in the same directory. README.md is the richer
+// one — it carries a status column — so it is the one kept.
+//
+// Also runnable directly:  node sync-adr-toc.cjs --write
+//
+// Fails open at exit 0 on any problem — a hook must never block a session.
+
+const fs = require('fs');
+const path = require('path');
+
+const root = process.env.CLAUDE_PROJECT_DIR || process.cwd();
+const ADR_FILE = /^\d+-.*\.md$/;
+
+// The tool reads its directory from a committed `.adr-dir` (default `doc/adr`).
+function adrDir() {
+  try {
+    return fs.readFileSync(path.join(root, '.adr-dir'), 'utf8').trim() || 'doc/adr';
+  } catch {
+    return 'doc/adr';
+  }
+}
+
+// Was this Bash call an adr command? (`adr new -q …` counts; a hyphenated name
+// such as `adr-new` does not, hence the required whitespace.)
+function isAdrCommand() {
+  try {
+    const payload = JSON.parse(fs.readFileSync(0, 'utf8'));
+    return /\badr\s/.test(payload?.tool_input?.command || '');
+  } catch {
+    return false;
+  }
+}
+
+// First line, minus the leading "# " and the tool's own number prefix — the
+// template renders "# 12: Use X over Y" for 0012-….md. Dropping the unpadded 12
+// keeps the padded
+// stem in the ADR column the only number in the file, so nothing here can be
+// mistaken for the stem `adr new -s` expects.
+function title(body, fallback) {
+  const first = body.split(/\r?\n/)[0] || '';
+  return first.replace(/^#\s*(\d+\s*[:.]\s*)?/, '').trim() || fallback;
+}
+
+// First non-empty line of the `## Status` section. The tool maintains this:
+// "Accepted", or "Superseded by [4: …](0004-….md)" after `adr new -s`.
+function status(body) {
+  const lines = body.split(/\r?\n/);
+  const start = lines.findIndex((l) => /^##\s+Status\s*$/i.test(l));
+  if (start === -1) return 'Unknown';
+  for (const line of lines.slice(start + 1)) {
+    if (/^##\s/.test(line)) break;
+    if (line.trim()) return line.trim();
+  }
+  return 'Unknown';
+}
+
+function render(dir, files) {
+  const rows = files.map((name) => {
+    const body = fs.readFileSync(path.join(root, dir, name), 'utf8');
+    const stem = name.replace(/\.md$/, ''); // padded — what `adr new -s` needs
+    const cell = (s) => s.replace(/\|/g, '\\|'); // never break the table
+    return `| [${stem}](./${name}) | ${cell(title(body, stem))} | ${cell(status(body))} |`;
+  });
+
+  return [
+    '# Architecture Decision Records',
+    '',
+    'Generated from the ADR files by the `sync-adr-toc` hook — do not hand-edit.',
+    '',
+    '| ADR | Title | Status |',
+    '|-----|-------|--------|',
+    ...rows,
+    '',
+  ].join('\n');
+}
+
+// The tool writes its own table of contents to `<adr-dir>/decisions.md` on every
+// `adr new`. README.md supersedes it, so drop it rather than commit two indexes
+// that have to agree with each other.
+//
+// Only ever deletes a file that is recognisably that generated index: the exact
+// heading it writes, followed by nothing but link lines. `decisions.md` is an
+// ordinary name for a hand-written decision log, and deleting one of those on
+// the next `adr new` would destroy work no backup covers.
+const GENERATED_TOC = /^# Table of Contents\r?\n\r?\n(?:- \[[^\]]*\]\([^)]*\)\r?\n?)*$/;
+
+function removeRedundantToc(dir) {
+  const stale = path.join(root, dir, 'decisions.md');
+  try {
+    if (!fs.existsSync(stale)) return;
+    if (!GENERATED_TOC.test(fs.readFileSync(stale, 'utf8'))) {
+      process.stdout.write(
+        `Left ${dir}/decisions.md alone — it is not the generated index. ` +
+          'README.md is the one this hook maintains.\n'
+      );
+      return;
+    }
+    fs.unlinkSync(stale);
+    process.stdout.write(`Removed ${dir}/decisions.md (README.md is the index).\n`);
+  } catch {
+    /* not worth failing the hook over */
+  }
+}
+
+function main() {
+  if (!process.argv.includes('--write') && !isAdrCommand()) return;
+
+  const dir = adrDir();
+  removeRedundantToc(dir);
+  const files = fs
+    .readdirSync(path.join(root, dir))
+    .filter((f) => ADR_FILE.test(f))
+    .sort();
+  if (files.length === 0) return;
+
+  const target = path.join(root, dir, 'README.md');
+  const next = render(dir, files);
+
+  let current = null;
+  try {
+    current = fs.readFileSync(target, 'utf8');
+  } catch {
+    /* no index yet */
+  }
+  if (current === next) return; // already in sync
+
+  fs.writeFileSync(target, next);
+  process.stdout.write(`Regenerated ${dir}/README.md (${files.length} ADRs).\n`);
+}
+
+try {
+  main();
+} catch {
+  /* fail open */
+}
+process.exit(0);

package/template/.claude/skills/adr-writer/SKILL.md

@@ -1,46 +1,48 @@
----
-name: adr-writer
-description: Write and manage Architecture Decision Records (ADRs) following structured methodology with proper numbering, status tracking, governance, and conversational writing style.
----
-
-# ADR Writer
-
-Write Architecture Decision Records — the log of "architecturally significant" decisions affecting structure, non-functional characteristics, dependencies, interfaces, or construction techniques.
-
-**Scope:** Create, update, and supersede ADRs. Does NOT implement the decisions themselves.
-
-## Workflow
-
-1. **Read the context.** `docs/architecture-characteristics.md` — every Decision is justified against these, with trade-offs framed as which are favored vs. sacrificed. If it is missing, read `.claude/skills/architecture-characteristic-writer/SKILL.md` and use its questions to gather the driving characteristics from the user before going on. Then read `docs/adr/README.md` for the decisions already recorded — one ADR's Consequences are often the next one's Context.
-
-2. **Research before suggesting** — REQUIRED whenever a technology, vendor, product, version, or price is in play. Invoke `/research`; never propose options from memory, it goes stale. Confirm each option still exists and is supported today. Report in one pass, source and verdict per option.
-
-3. **Challenge the proposal — be harsh.** REQUIRED whenever the user proposes a specific option. Do not rubber-stamp it:
-   - **State why** — the concrete reasons, not familiarity or preference.
-   - **Score it against each driving and implicit characteristic** — does it *serve*, *ignore*, or *actively harm* it?
-   - **Give a verdict** — suitable / suitable-with-trade-offs / unsuitable. If it conflicts with a driving characteristic, say so plainly and recommend the better fit, even when that is not what was asked for.
-   - Continue only once it survives, or the user overrides knowing the trade-off — record that override as a Consequence.
-
-4. **Create the file.** From the repo root:
-   ```bash
-   adr new -- "Use X for Z"
-   ```
-   - Use the path `adr new` prints — never guess the number or slug.
-   - Superseding — always the full padded stem, never a bare number (`-s 2` matches the first path containing "2", possibly the wrong file). Writes the cross-link into both ADRs' `## Status`:
-     ```bash
-     adr new -s 0002-use-mysql-for-persistence -- "…"
-     ```
-
-5. **Write the ADR.** Replace every `{…}` placeholder — nothing else. `adr new` has already written the number, title, `Accepted` status, and any supersede/link lines; leave all of them alone. Section Guidance below covers what each section needs. An Accepted ADR is immutable — to change it, write a new ADR that supersedes it.
-
-## Writing Style
-
-- **Calibrate against `references/adr-example.md`** — re-read it before drafting; it is the target for terseness. Cut any section markedly longer than its equivalent.
-- **Match `references/adr-template.md` exactly.** Use only the sections it defines. No invented sections or fields; research citations go inline in the Decision.
-
-## Section Guidance
-
-- **Title** — reveal the *decision*, not the topic: "Use X for Z". Bad: "Gmail Polling for Ingestion". Good: "Use Cloud Scheduler Polling for Gmail Ingestion".
-- **Context** — value-neutral, tensions explicit. No alternatives here, and no notes about what is *not* being decided.
-- **Decision** — record the WHY, not the HOW — omit libraries, drivers, and wiring.
-- **Consequences** — both `### Positive` and `### Negative` required. Consider team, infrastructure, cost, and one-way doors.
+---
+name: adr-writer
+description: Write and manage Architecture Decision Records (ADRs) following structured methodology with proper numbering, status tracking, governance, and conversational writing style.
+---
+
+# ADR Writer
+
+Write Architecture Decision Records — the log of "architecturally significant" decisions affecting structure, non-functional characteristics, dependencies, interfaces, or construction techniques.
+
+**Scope:** Create, update, and supersede ADRs. Does NOT implement the decisions themselves.
+
+## Workflow
+
+1. **Read the context.** `docs/architecture-characteristics.md` — every Decision is justified against these, with trade-offs framed as which are favored vs. sacrificed. If it is missing, read `.claude/skills/architecture-characteristic-writer/SKILL.md` and use its questions to gather the driving characteristics from the user before going on. Then read `docs/adr/README.md` for the decisions already recorded — one ADR's Consequences are often the next one's Context.
+
+2. **Research before suggesting** — REQUIRED whenever a technology, vendor, product, version, or price is in play. Invoke `/research`; never propose options from memory, it goes stale. Confirm each option still exists and is supported today. Report in one pass, source and verdict per option.
+
+3. **Challenge the proposal — be harsh.** REQUIRED whenever the user proposes a specific option. Do not rubber-stamp it:
+   - **State why** — the concrete reasons, not familiarity or preference.
+   - **Score it against each driving and implicit characteristic** — does it *serve*, *ignore*, or *actively harm* it?
+   - **Give a verdict** — suitable / suitable-with-trade-offs / unsuitable. If it conflicts with a driving characteristic, say so plainly and recommend the better fit, even when that is not what was asked for.
+   - Continue only once it survives, or the user overrides knowing the trade-off — record that override as a Consequence.
+
+4. **Create the file.** From the repo root:
+   ```bash
+   adr new -q -- "Use X for Z"
+   ```
+   - `-q` is required on every call. Without it an ambiguous `-s` prompts on stdin and hangs the session; with it, that case errors instead.
+   - `adr new` prints nothing. Run `adr list` (last line is the new file) or read `docs/adr/README.md` — never guess the number or slug.
+   - Never use uppercase `STATUS` in a title. It is substituted after the title is inserted, so "Use STATUS codes" silently becomes "Use Accepted codes" and leaves the real Status unset. Lowercase is fine.
+   - Superseding — always the full padded stem, never a bare number. `-s` matches by substring: with `-q` an *ambiguous* match errors, but a single *wrong* match still succeeds silently. Writes the cross-link into both ADRs' `## Status`:
+     ```bash
+     adr new -q -s 0002-use-mysql-for-persistence -- "…"
+     ```
+
+5. **Write the ADR.** Replace every `{…}` placeholder — nothing else. `adr new` has already written the number, title, `Accepted` status, and any supersede/link lines; leave all of them alone. Section Guidance below covers what each section needs. An Accepted ADR is immutable — to change it, write a new ADR that supersedes it.
+
+## Writing Style
+
+- **Calibrate against `references/adr-example.md`** — re-read it before drafting; it is the target for terseness. Cut any section markedly longer than its equivalent.
+- **Match `references/adr-template.md` exactly.** Use only the sections it defines. No invented sections or fields; research citations go inline in the Decision.
+
+## Section Guidance
+
+- **Title** — reveal the *decision*, not the topic: "Use X for Z". Bad: "Gmail Polling for Ingestion". Good: "Use Cloud Scheduler Polling for Gmail Ingestion".
+- **Context** — value-neutral, tensions explicit. No alternatives here, and no notes about what is *not* being decided.
+- **Decision** — record the WHY, not the HOW — omit libraries, drivers, and wiring.
+- **Consequences** — both `### Positive` and `### Negative` required. Consider team, infrastructure, cost, and one-way doors.

package/template/.claude/hooks/inject-adr-env.cjs

@@ -1,82 +0,0 @@
-#!/usr/bin/env node
-// PreToolUse guard: inject the env vars every `adr new` call needs, so the
-// agent can write a bare `adr new -- "Title"` and get a correct command.
-//
-//   VISUAL=true EDITOR=true  — without them an ambient EDITOR opens an
-//     interactive editor and hangs the session (adr-new always shells out to
-//     $VISUAL/$EDITOR to "edit" the new file).
-//   ADR_TEMPLATE=...adr-template.md — points adr-new at this skill's template
-//     instead of adr-tools' own bundled default.
-//
-// Uses PreToolUse's `updatedInput` (not just allow/deny) to rewrite the
-// command before it runs. Only fills in vars that are missing — an assignment
-// the agent already wrote (any value, even a deliberately different template)
-// is left alone. Silent no-op if all three are already present.
-//
-// Safety: a title that happens to contain the literal text "adr new" could
-// make the matching regex fire inside a quoted string. Guarded by counting
-// quote characters before each match — an odd count means "inside an open
-// string", and that occurrence is left untouched rather than risk corrupting
-// the title.
-
-const fs = require('fs');
-
-const DEFAULTS = [
-  ['ADR_TEMPLATE', '.claude/skills/adr-writer/references/adr-template.md'],
-  ['VISUAL', 'true'],
-  ['EDITOR', 'true'],
-];
-
-// boundary/start, leading whitespace, any existing KEY=value prefix, `adr new`.
-const ADR_NEW_RE = /(^|[;&|\n])([ \t]*)((?:[A-Za-z_][A-Za-z0-9_]*=(?:'[^']*'|"[^"]*"|\S*)[ \t]+)*)(adr[ \t]+new\b)/g;
-
-function existingVars(prefix) {
-  const names = new Set();
-  const re = /([A-Za-z_][A-Za-z0-9_]*)=/g;
-  let m;
-  while ((m = re.exec(prefix))) names.add(m[1]);
-  return names;
-}
-
-function isInsideQuotes(command, index) {
-  const before = command.slice(0, index);
-  const dq = (before.match(/"/g) || []).length;
-  const sq = (before.match(/'/g) || []).length;
-  return dq % 2 === 1 || sq % 2 === 1;
-}
-
-function main() {
-  const payload = JSON.parse(fs.readFileSync(0, 'utf8'));
-  const command = payload?.tool_input?.command || '';
-  if (!/\badr\s+new\b/.test(command)) return;
-
-  let changed = false;
-  const updated = command.replace(ADR_NEW_RE, (whole, boundary, ws, existingPrefix, adrNew, offset) => {
-    if (isInsideQuotes(command, offset)) return whole;
-    const present = existingVars(existingPrefix);
-    const missing = DEFAULTS.filter(([name]) => !present.has(name));
-    if (missing.length === 0) return whole;
-    changed = true;
-    const injected = missing.map(([name, value]) => `${name}=${value}`).join(' ');
-    return `${boundary}${ws}${injected} ${existingPrefix}${adrNew}`;
-  });
-  if (!changed) return;
-
-  process.stdout.write(
-    JSON.stringify({
-      systemMessage: 'Added missing VISUAL/EDITOR/ADR_TEMPLATE env vars before `adr new`.',
-      hookSpecificOutput: {
-        hookEventName: 'PreToolUse',
-        permissionDecision: 'allow',
-        updatedInput: { command: updated },
-      },
-    })
-  );
-}
-
-try {
-  main();
-} catch {
-  /* fail open: leave the command untouched */
-}
-process.exit(0);

package/template/dotnet-scaffold/src/__SolutionName__.Api/obj/Debug/net10.0/.NETCoreApp,Version=v10.0.AssemblyAttributes.cs

@@ -1,4 +0,0 @@
-// <autogenerated />
-using System;
-using System.Reflection;
-[assembly: global::System.Runtime.Versioning.TargetFrameworkAttribute(".NETCoreApp,Version=v10.0", FrameworkDisplayName = ".NET 10.0")]

package/template/dotnet-scaffold/src/__SolutionName__.Api/obj/Debug/net10.0/__SolutionName__.Api.AssemblyInfo.cs

@@ -1,22 +0,0 @@
-//------------------------------------------------------------------------------
-// <auto-generated>
-//     This code was generated by a tool.
-//
-//     Changes to this file may cause incorrect behavior and will be lost if
-//     the code is regenerated.
-// </auto-generated>
-//------------------------------------------------------------------------------
-
-using System;
-using System.Reflection;
-
-[assembly: System.Reflection.AssemblyCompanyAttribute("__SolutionName__.Api")]
-[assembly: System.Reflection.AssemblyConfigurationAttribute("Debug")]
-[assembly: System.Reflection.AssemblyFileVersionAttribute("1.0.0.0")]
-[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+b8b9e4e3c73f140ae7b06d954c63afb26d78c687")]
-[assembly: System.Reflection.AssemblyProductAttribute("__SolutionName__.Api")]
-[assembly: System.Reflection.AssemblyTitleAttribute("__SolutionName__.Api")]
-[assembly: System.Reflection.AssemblyVersionAttribute("1.0.0.0")]
-
-// Generated by the MSBuild WriteCodeFragment class.
-

package/template/dotnet-scaffold/src/__SolutionName__.Api/obj/Debug/net10.0/__SolutionName__.Api.AssemblyInfoInputs.cache

@@ -1 +0,0 @@
-922649250c3cbe1ea76366976311f3d76bad60fe3f765d7bf9d3d9b685476c9f

package/template/dotnet-scaffold/src/__SolutionName__.Api/obj/Debug/net10.0/__SolutionName__.Api.GeneratedMSBuildEditorConfig.editorconfig

@@ -1,23 +0,0 @@
-is_global = true
-build_property.TargetFramework = net10.0
-build_property.TargetFrameworkIdentifier = .NETCoreApp
-build_property.TargetFrameworkVersion = v10.0
-build_property.TargetPlatformMinVersion = 
-build_property.UsingMicrosoftNETSdkWeb = true
-build_property.ProjectTypeGuids = 
-build_property.InvariantGlobalization = 
-build_property.PlatformNeutralAssembly = 
-build_property.EnforceExtendedAnalyzerRules = 
-build_property._SupportedPlatformList = Linux,macOS,Windows
-build_property.RootNamespace = __SolutionName__.Api
-build_property.RootNamespace = __SolutionName__.Api
-build_property.ProjectDir = C:\Projects2\Personal\musketeer\template\dotnet-scaffold\src\__SolutionName__.Api\
-build_property.EnableComHosting = 
-build_property.EnableGeneratedComInterfaceComImportInterop = 
-build_property.RazorLangVersion = 9.0
-build_property.SupportLocalizedComponentNames = 
-build_property.GenerateRazorMetadataSourceChecksumAttributes = 
-build_property.MSBuildProjectDirectory = C:\Projects2\Personal\musketeer\template\dotnet-scaffold\src\__SolutionName__.Api
-build_property._RazorSourceGeneratorDebug = 
-build_property.EffectiveAnalysisLevelStyle = 10.0
-build_property.EnableCodeStyleSeverity = 

package/template/dotnet-scaffold/src/__SolutionName__.Api/obj/Debug/net10.0/__SolutionName__.Api.GlobalUsings.g.cs

@@ -1,17 +0,0 @@
-// <auto-generated/>
-global using Microsoft.AspNetCore.Builder;
-global using Microsoft.AspNetCore.Hosting;
-global using Microsoft.AspNetCore.Http;
-global using Microsoft.AspNetCore.Routing;
-global using Microsoft.Extensions.Configuration;
-global using Microsoft.Extensions.DependencyInjection;
-global using Microsoft.Extensions.Hosting;
-global using Microsoft.Extensions.Logging;
-global using System;
-global using System.Collections.Generic;
-global using System.IO;
-global using System.Linq;
-global using System.Net.Http;
-global using System.Net.Http.Json;
-global using System.Threading;
-global using System.Threading.Tasks;