@erclx/canon 5.30.0 → 5.32.0

package/src/commands/autoship.ts

@@ -1,129 +0,0 @@
-import type { Command } from 'commander'
-import {
-  type ClassifyRefusal,
-  type FailedTest,
-  classifyChanges,
-} from '@/autoship/classify'
-import { intro, logInfo, logStep, logWarn, outro, plural } from '@/cli/ui'
-
-interface ClassifyOptions {
-  readonly json?: boolean
-}
-
-/** What a reader does about the one way the classification produced no reading. */
-const REFUSALS: Record<ClassifyRefusal, string> = {
-  'no-changes':
-    'No changed files were supplied, so there is no set to classify. Pass the set the ship chain already computed.',
-}
-
-/** Why each failed test sends the branch to review, in the reader's terms. */
-const TESTS: Record<FailedTest, string> = {
-  extension: 'is not prose, so this branch carries a change review reads',
-  'behavior-path':
-    'sits under a behavior path, so its prose states what an agent does',
-}
-
-export function register(program: Command): void {
-  const autoship = program
-    .command('autoship')
-    .description('Answer the decisions the ship chain used to make in prose')
-    .helpOption('-h, --help', 'Show this help message')
-
-  autoship
-    .command('classify')
-    .description('Decide whether a changed set needs the review pass')
-    .argument('[paths...]', 'Changed set to classify, as names')
-    .helpOption('-h, --help', 'Show this help message')
-    .option('--json', 'Add a machine-readable record on stdout')
-    .addHelpText(
-      'after',
-      [
-        '',
-        'Reads names only and never touches git, so the caller hands over the',
-        'set it already computed rather than paying for a second baseline that',
-        'can disagree with the first.',
-        '',
-        'The skip needs both tests to pass. Every name reads as prose, and no',
-        'name sits under a behavior path, which is where markdown states what',
-        'an agent does. One behavior file sends the whole branch to review,',
-        'since documentation shipped beside a behavior change does not cancel',
-        'it.',
-        '',
-        'An empty set refuses rather than skipping. Both tests are universally',
-        'quantified, so an empty set satisfies them vacuously, and reading that',
-        'as prose-only would route a branch past review for producing no output.',
-        '',
-        'Exit codes:',
-        '  0  prose-only, so the review pass can be skipped',
-        '  1  refused, with the reason on stderr or in the JSON record',
-        '  2  review is needed, with the file and the test it failed named',
-        '',
-        'Examples:',
-        '  canon autoship classify docs/index.md README.md',
-        '  canon autoship classify --json .claude/skills/deploy-check/SKILL.md',
-        '',
-      ].join('\n'),
-    )
-    .action((paths: string[], opts: ClassifyOptions) => {
-      process.exitCode = runClassify(paths, opts)
-    })
-}
-
-function runClassify(paths: string[], opts: ClassifyOptions): number {
-  const emitJson = opts.json ?? false
-  const result = classifyChanges(paths)
-
-  intro('canon autoship classify')
-
-  // The frame renders on stderr in both modes and the record goes to stdout
-  // alone, so an operator reading the terminal sees the refusal rather than a
-  // command that appeared to do nothing.
-  if (result.kind === 'refused') {
-    logStep('Refused')
-    logWarn(REFUSALS[result.reason])
-    outro()
-
-    if (emitJson) {
-      process.stdout.write(
-        `${JSON.stringify({
-          decision: 'refused',
-          reason: result.reason,
-          message: REFUSALS[result.reason],
-        })}\n`,
-      )
-    }
-    return 1
-  }
-
-  logStep('Scope')
-  logInfo(`${plural(paths.length, 'path')} supplied by the caller`)
-
-  if (result.kind === 'skip') {
-    logStep('Prose only')
-    logInfo('every path reads as prose and none states agent behavior')
-    outro()
-
-    if (emitJson) {
-      process.stdout.write(
-        `${JSON.stringify({ decision: 'skip', changed: paths })}\n`,
-      )
-    }
-    return 0
-  }
-
-  logStep('Review')
-  logWarn(`${result.file} ${TESTS[result.test]}.`)
-  outro()
-
-  if (emitJson) {
-    process.stdout.write(
-      `${JSON.stringify({
-        decision: 'review',
-        test: result.test,
-        file: result.file,
-        changed: paths,
-      })}\n`,
-    )
-  }
-  return 2
-}

package/src/commands/repo.ts

@@ -1,393 +0,0 @@
-import { resolve } from 'node:path'
-import type { Command } from 'commander'
-import { execa } from 'execa'
-import { gitEnv } from '@/git/env'
-import {
-  compareMetadata,
-  type CurrentMetadata,
-  isValidTopic,
-  type MetadataDiff,
-  proposeMetadata,
-} from '@/repo/metadata'
-import {
-  intro,
-  logAdd,
-  logInfo,
-  logRemove,
-  logStep,
-  logWarn,
-  outro,
-} from '@/cli/ui'
-
-const GH_TIMEOUT_MS = 30_000
-
-type SourceRefusal = 'gh-missing' | 'gh-failed'
-type ApplyRefusal =
-  | SourceRefusal
-  | 'no-changes'
-  | 'empty-topics'
-  | 'invalid-topics'
-  | 'wrong-repo'
-
-const REFUSALS: Record<ApplyRefusal, string> = {
-  'gh-missing': 'gh is not on the path, so no remote could be read.',
-  'gh-failed':
-    'gh could not read this repository. Check the remote and gh auth status.',
-  'no-changes':
-    'Nothing to apply. Pass --description, --homepage, or --topics with the answered value.',
-  'empty-topics':
-    '--topics carried no usable topic, and this command never reads that as a request to remove every topic the remote carries. Pass the full desired set, or omit the flag to leave topics untouched.',
-  'invalid-topics':
-    '--topics carried an entry GitHub does not accept as a topic (lowercase letters, digits, and internal hyphens only). This command never narrows the desired set silently, since dropping it changes what the remaining diff removes.',
-  'wrong-repo':
-    'The remote --root resolved to is not the one --repo named. This command never writes to a repository the invocation did not explicitly confirm.',
-}
-
-interface ProposeOptions {
-  readonly root?: string
-  readonly json?: boolean
-}
-
-interface ApplyOptions {
-  readonly description?: string
-  readonly homepage?: string
-  readonly topics?: string
-  readonly repo: string
-  readonly root?: string
-  readonly json?: boolean
-}
-
-type CurrentRead =
-  | {
-      readonly kind: 'read'
-      readonly current: CurrentMetadata
-      readonly nameWithOwner: string
-    }
-  | { readonly kind: 'refused'; readonly reason: SourceRefusal }
-
-/** Reads what the remote already carries. The one network call this domain makes to read. */
-async function readCurrent(cwd: string): Promise<CurrentRead> {
-  if (Bun.which('gh') === null) return { kind: 'refused', reason: 'gh-missing' }
-
-  try {
-    const result = await execa(
-      'gh',
-      [
-        'repo',
-        'view',
-        '--json',
-        'description,homepageUrl,repositoryTopics,nameWithOwner',
-      ],
-      { cwd, timeout: GH_TIMEOUT_MS, env: gitEnv(), extendEnv: false },
-    )
-    const row = JSON.parse(result.stdout) as {
-      description?: string
-      homepageUrl?: string
-      repositoryTopics?: readonly { name: string }[]
-      nameWithOwner?: string
-    }
-    return {
-      kind: 'read',
-      current: {
-        description: row.description ?? '',
-        homepage: row.homepageUrl ?? '',
-        topics: (row.repositoryTopics ?? []).map((topic) => topic.name),
-      },
-      nameWithOwner: row.nameWithOwner ?? '',
-    }
-  } catch {
-    return { kind: 'refused', reason: 'gh-failed' }
-  }
-}
-
-export function register(program: Command): void {
-  const repo = program
-    .command('repo')
-    .description('Read and write this repository’s own remote metadata')
-    .helpOption('-h, --help', 'Show this help message')
-
-  const metadata = repo
-    .command('metadata')
-    .description('The About description, homepage, and topics GitHub shows')
-    .helpOption('-h, --help', 'Show this help message')
-
-  metadata
-    .command('propose')
-    .description(
-      'Compare a locally computed description, homepage, and topic set against the remote',
-    )
-    .helpOption('-h, --help', 'Show this help message')
-    .option('--root <path>', 'Repository to read, defaulting to the cwd')
-    .option('--json', 'Add a machine-readable record on stdout')
-    .addHelpText(
-      'after',
-      [
-        '',
-        "Computes from the tree alone: a description from the README's opening",
-        'line past its title and badges, and a homepage and topics from',
-        "package.json's own homepage and keywords fields. A field neither file",
-        'declares is never diffed, so an undeclared topic set never reads as a',
-        'proposal to clear what the remote already carries.',
-        '',
-        'This is the read half. It never writes. Run `canon repo metadata apply`',
-        'with the answered fields once a proposal is worth taking.',
-        '',
-        'Exit codes:',
-        '  0  the remote already carries what this run computed',
-        '  1  refused, with the reason on stderr or in the JSON record',
-        '  2  a difference was found',
-        '',
-        'Examples:',
-        '  canon repo metadata propose',
-        '  canon repo metadata propose --json',
-        '',
-      ].join('\n'),
-    )
-    .action(async (opts: ProposeOptions) => {
-      process.exitCode = await runPropose(opts)
-    })
-
-  metadata
-    .command('apply')
-    .description(
-      'Write an explicitly supplied description, homepage, or topic set to the remote',
-    )
-    .helpOption('-h, --help', 'Show this help message')
-    .requiredOption(
-      '--repo <owner/name>',
-      'Repository this run is allowed to write to, checked against what gh resolves',
-    )
-    .option('--description <text>', 'About text to write')
-    .option('--homepage <url>', 'Homepage URL to write')
-    .option(
-      '--topics <list>',
-      'Comma-separated topic set to write, replacing what the remote carries',
-    )
-    .option('--root <path>', 'Repository to read, defaulting to the cwd')
-    .option('--json', 'Add a machine-readable record on stdout')
-    .addHelpText(
-      'after',
-      [
-        '',
-        'Takes the answered value for each field directly rather than',
-        're-running propose, so a change this writes is always one a person or',
-        'a second invocation already read and confirmed. --topics is the full',
-        'desired set. gh only takes an add and a remove list, so this reads the',
-        'current set once to compute both. An entry shaped like something other',
-        'than a GitHub topic refuses the whole run rather than being dropped.',
-        '',
-        '--repo is required and takes no default, since --root silently',
-        'resolving to the caller’s own cwd is what turned a verification run',
-        'against this exact command into a live write against a public remote.',
-        'The value is checked against what gh resolves for --root, so the',
-        'write refuses on any repository the invocation did not name aloud.',
-        '',
-        'Exit codes:',
-        '  0  the write succeeded, or the remote already matched',
-        '  1  refused, with the reason on stderr or in the JSON record',
-        '',
-        'Examples:',
-        '  canon repo metadata apply --repo erclx/canon --description "One source for conventions."',
-        '  canon repo metadata apply --repo erclx/canon --topics cli-tool,governance,standards',
-        '',
-      ].join('\n'),
-    )
-    .action(async (opts: ApplyOptions) => {
-      process.exitCode = await runApply(opts)
-    })
-}
-
-function refuse(
-  reason: ApplyRefusal,
-  emitJson: boolean,
-  root: string,
-  detail?: string,
-): number {
-  const message =
-    detail === undefined ? REFUSALS[reason] : `${REFUSALS[reason]} ${detail}`
-
-  logStep('Refused')
-  logWarn(message)
-  outro()
-
-  if (emitJson) {
-    process.stdout.write(`${JSON.stringify({ root, reason, message })}\n`)
-  }
-  return 1
-}
-
-async function runPropose(opts: ProposeOptions): Promise<number> {
-  const root = resolve(opts.root ?? process.cwd())
-  const emitJson = opts.json ?? false
-
-  intro('canon repo metadata propose')
-
-  const [proposal, currentRead] = await Promise.all([
-    proposeMetadata(root),
-    readCurrent(root),
-  ])
-
-  if (currentRead.kind === 'refused') {
-    return refuse(currentRead.reason, emitJson, root)
-  }
-
-  const diff: MetadataDiff = compareMetadata(currentRead.current, proposal)
-  const changed = Object.keys(diff).length > 0
-
-  logStep('Repository')
-  logInfo(currentRead.nameWithOwner)
-
-  logStep('Computed')
-  if (
-    proposal.description === undefined &&
-    proposal.homepage === undefined &&
-    proposal.topics === undefined
-  ) {
-    logInfo('nothing local resolved a description, a homepage, or topics')
-  } else {
-    if (proposal.description !== undefined) {
-      logInfo(`description: ${proposal.description}`)
-    }
-    if (proposal.homepage !== undefined)
-      logInfo(`homepage: ${proposal.homepage}`)
-    if (proposal.topics !== undefined) {
-      logInfo(`topics: ${proposal.topics.join(', ')}`)
-    }
-  }
-
-  logStep(changed ? 'Difference' : 'No difference')
-  if (!changed) {
-    logInfo('the remote already carries what this run computed')
-  } else {
-    if (diff.description !== undefined) {
-      logWarn(
-        `description: "${diff.description.current}" → "${diff.description.proposed}"`,
-      )
-    }
-    if (diff.homepage !== undefined) {
-      logWarn(
-        `homepage: "${diff.homepage.current}" → "${diff.homepage.proposed}"`,
-      )
-    }
-    if (diff.topics !== undefined) {
-      for (const topic of diff.topics.added) logAdd(`topic: ${topic}`)
-      for (const topic of diff.topics.removed) logRemove(`topic: ${topic}`)
-    }
-    logInfo(
-      'Run `canon repo metadata apply` with the answered fields. This run writes nothing.',
-    )
-  }
-
-  outro()
-
-  if (emitJson) {
-    process.stdout.write(
-      `${JSON.stringify({
-        root,
-        repo: currentRead.nameWithOwner,
-        current: currentRead.current,
-        proposal,
-        diff,
-      })}\n`,
-    )
-  }
-
-  return changed ? 2 : 0
-}
-
-async function runApply(opts: ApplyOptions): Promise<number> {
-  const root = resolve(opts.root ?? process.cwd())
-  const emitJson = opts.json ?? false
-
-  intro('canon repo metadata apply')
-
-  if (
-    opts.description === undefined &&
-    opts.homepage === undefined &&
-    opts.topics === undefined
-  ) {
-    return refuse('no-changes', emitJson, root)
-  }
-
-  let desired: ReadonlySet<string> | undefined
-  if (opts.topics !== undefined) {
-    const entries = opts.topics
-      .split(',')
-      .map((topic) => topic.trim().toLowerCase())
-      .filter((topic) => topic !== '')
-    if (entries.length === 0) return refuse('empty-topics', emitJson, root)
-
-    const invalid = entries.filter((topic) => !isValidTopic(topic))
-    if (invalid.length > 0) {
-      return refuse(
-        'invalid-topics',
-        emitJson,
-        root,
-        `Invalid: ${invalid.join(', ')}`,
-      )
-    }
-
-    desired = new Set(entries)
-  }
-
-  // Read ahead of every write, never only for --topics, since this is the
-  // one call that confirms --repo names the remote --root actually resolved
-  // to. A write skipping it on a description-only or homepage-only run would
-  // reopen the gap --repo exists to close.
-  const currentRead = await readCurrent(root)
-  if (currentRead.kind === 'refused')
-    return refuse(currentRead.reason, emitJson, root)
-  if (currentRead.nameWithOwner !== opts.repo) {
-    return refuse('wrong-repo', emitJson, root)
-  }
-
-  const args = ['repo', 'edit']
-  if (opts.description !== undefined)
-    args.push('--description', opts.description)
-  if (opts.homepage !== undefined) args.push('--homepage', opts.homepage)
-
-  if (desired !== undefined) {
-    const currentSet = new Set(currentRead.current.topics)
-    const toAdd = [...desired].filter((topic) => !currentSet.has(topic))
-    const toRemove = currentRead.current.topics.filter(
-      (topic) => !desired.has(topic),
-    )
-
-    if (toAdd.length > 0) args.push('--add-topic', toAdd.join(','))
-    if (toRemove.length > 0) args.push('--remove-topic', toRemove.join(','))
-  }
-
-  if (args.length === 2) {
-    logStep('Applied')
-    logInfo('the remote already matches every field supplied. Nothing written.')
-    outro()
-
-    if (emitJson) {
-      process.stdout.write(`${JSON.stringify({ root, written: false })}\n`)
-    }
-    return 0
-  }
-
-  if (Bun.which('gh') === null) return refuse('gh-missing', emitJson, root)
-
-  try {
-    await execa('gh', args, {
-      cwd: root,
-      timeout: GH_TIMEOUT_MS,
-      env: gitEnv(),
-      extendEnv: false,
-    })
-  } catch {
-    return refuse('gh-failed', emitJson, root)
-  }
-
-  logStep('Applied')
-  logInfo('gh repo edit wrote the supplied fields to the remote.')
-  outro()
-
-  if (emitJson) {
-    process.stdout.write(`${JSON.stringify({ root, written: true })}\n`)
-  }
-
-  return 0
-}