superwiki 0.1.2 → 0.1.4

package/skills/sw-init/scripts/init.mjs

@@ -1,118 +1,197 @@
 #!/usr/bin/env node
-// Scaffolds docs/ as a Superwiki vault. Safe to re-run: user content is kept, tool files are refreshed.
-import { readFileSync, writeFileSync, existsSync, mkdirSync, readdirSync } from 'node:fs';
+// Scaffolds docs/ as a Superwiki vault. Safe to re-run: user content is kept, tool files are
+// replaced with this version, and the report says which was which.
+import { existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from 'node:fs';
 import { basename, dirname, join, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
 
-const assets = join(dirname(fileURLToPath(import.meta.url)), '..', 'assets');
-const argv = process.argv.slice(2);
-const opt = name => { const i = argv.indexOf(name); return i >= 0 ? argv[i + 1] : undefined; };
+const ASSETS = join(dirname(fileURLToPath(import.meta.url)), '..', 'assets');
+const AREA_ID = /^[A-Za-z][A-Za-z0-9]*$/;
+const MANAGED_BLOCK = /<!-- sw:start[\s\S]*?<!-- sw:end -->/;
+// What a vault consists of at the top of docs/. Anything else there belongs to someone else.
+const VAULT_ENTRIES = ['index.md', 'log.md', 'raw', 'wiki', 'tasks', 'plans', 'viewer.html'];
+const ROLES = ['plan', 'implement', 'review'];
 
-if (argv.includes('--help')) {
-  console.log(`init.mjs [--root <dir>] [--tasks | --no-tasks] [--areas "M=Mobile,B=Backend"]
+const HELP = `init.mjs [--root <dir>] [--tasks | --no-tasks] [--areas "M=Mobile,B=Backend"]
 
   --root      project folder (default: current directory)
   --tasks     add the task module (docs/tasks, docs/plans)
   --no-tasks  wiki only
-  --areas     task id prefixes and their names (default: T=Tasks)`);
-  process.exit(0);
+  --areas     task id prefixes and their names (default: T=Tasks)
+
+Run without --tasks or --no-tasks on an existing vault to upgrade it with its saved choices.`;
+
+class UsageError extends Error {}
+
+function parseArgs(argv) {
+  const options = { root: '.', tasks: null, areas: {}, help: false };
+  for (let i = 0; i < argv.length; i++) {
+    const arg = argv[i];
+    if (arg === '--help') options.help = true;
+    else if (arg === '--root') options.root = argv[++i] ?? '.';
+    else if (arg === '--tasks') options.tasks = true;
+    else if (arg === '--no-tasks') options.tasks = false;
+    else if (arg === '--areas') options.areas = parseAreas(argv[++i] ?? '');
+    else throw new UsageError(`unknown argument: ${arg}\n\n${HELP}`);
+  }
+  return options;
+}
+
+function parseAreas(text) {
+  const areas = {};
+  for (const pair of text.split(',').map(part => part.trim()).filter(Boolean)) {
+    const [id, ...name] = pair.split('=');
+    if (!AREA_ID.test(id)) throw new UsageError(`bad area id "${id}": letters and digits, starting with a letter`);
+    areas[id] = name.join('=').trim() || id;
+  }
+  return areas;
 }
 
-const root = resolve(opt('--root') || '.');
-const docs = join(root, 'docs');
-const dotdir = join(docs, '.sw');
-const configPath = join(dotdir, 'config.json');
-const previous = existsSync(configPath) ? JSON.parse(readFileSync(configPath, 'utf8')) : null;
-const tasks = argv.includes('--no-tasks') ? false : argv.includes('--tasks') ? true : previous?.tasks ?? null;
-if (tasks === null) {
-  console.error('say --tasks or --no-tasks');
-  process.exit(2);
+// Collects one report line per file or folder touched. The state of a tool-owned file is
+// judged by its content, so an upgrade that changes nothing says "unchanged".
+function createReport(root) {
+  const lines = [];
+  const rel = path => path.slice(root.length + 1);
+  const add = (state, text) => lines.push(`${state.padEnd(9)} ${text}`);
+  return {
+    lines,
+    folder(path) {
+      if (existsSync(path)) return;
+      mkdirSync(path, { recursive: true });
+      add('created', `${rel(path)}/`);
+    },
+    // User content: written once, never replaced.
+    keep(path, content) {
+      if (existsSync(path)) return add('kept', rel(path));
+      writeFileSync(path, content);
+      add('created', rel(path));
+    },
+    // Tool-owned content: always brought up to date.
+    write(path, content, label = rel(path)) {
+      const before = existsSync(path) ? readFileSync(path, 'utf8') : null;
+      if (before !== content) writeFileSync(path, content);
+      add(before === null ? 'created' : before === content ? 'unchanged' : 'updated', label);
+    },
+    copy(from, to) {
+      if (existsSync(from)) this.write(to, readFileSync(from, 'utf8'));
+      else add('missing', `${rel(to)} (not in this Superwiki build; report this to the user)`);
+    },
+    line: add,
+  };
 }
 
-const areas = {};
-for (const pair of (opt('--areas') || '').split(',').map(s => s.trim()).filter(Boolean)) {
-  const [id, ...name] = pair.split('=');
-  if (!/^[A-Za-z][A-Za-z0-9]*$/.test(id)) { console.error(`bad area id "${id}": letters and digits, starting with a letter`); process.exit(2); }
-  areas[id] = name.join('=').trim() || id;
+function readConfig(path) {
+  return existsSync(path) ? JSON.parse(readFileSync(path, 'utf8')) : null;
 }
 
-const today = new Date().toISOString().slice(0, 10);
-const report = [];
-const rel = p => p.slice(root.length + 1);
-const dir = p => { if (!existsSync(p)) { mkdirSync(p, { recursive: true }); report.push(`created   ${rel(p)}/`); } };
-const keep = (p, content) => {
-  if (existsSync(p)) { report.push(`kept      ${rel(p)}`); return false; }
-  writeFileSync(p, content);
-  report.push(`created   ${rel(p)}`);
-  return true;
-};
-// Writes a tool-owned file and reports created / updated / unchanged from the actual content.
-const write = (p, content, label = rel(p)) => {
-  const before = existsSync(p) ? readFileSync(p, 'utf8') : null;
-  if (before !== content) writeFileSync(p, content);
-  report.push(`${before === null ? 'created  ' : before === content ? 'unchanged' : 'updated  '} ${label}`);
-};
-const refresh = (from, to) => {
-  if (!existsSync(from)) { report.push(`missing   ${rel(to)} (not in this Superwiki build; report this to the user)`); return; }
-  write(to, readFileSync(from, 'utf8'));
-};
-
-const foreign = existsSync(docs) && !previous ? readdirSync(docs).filter(n => !n.startsWith('.')) : [];
-
-dir(docs);
-dir(join(docs, 'raw'));
-dir(join(docs, 'raw', 'assets'));
-dir(join(docs, 'wiki'));
-if (tasks) { dir(join(docs, 'tasks')); dir(join(docs, 'plans')); }
-dir(dotdir);
-dir(join(dotdir, 'templates'));
-// Git drops empty folders; the vault needs them to exist.
-for (const d of ['raw/assets', 'wiki', ...(tasks ? ['tasks', 'plans'] : [])]) {
-  const p = join(docs, d);
-  if (!readdirSync(p).length) writeFileSync(join(p, '.gitkeep'), '');
+function buildConfig(previous, { root, tasks, areas }) {
+  const models = Object.fromEntries(ROLES.map(role => [role, previous?.models?.[role] ?? {}]));
+  const chosenAreas = Object.keys(areas).length ? areas : previous?.areas ?? { T: 'Tasks' };
+  return {
+    version: 1,
+    name: previous?.name ?? basename(root),
+    tasks,
+    areas: tasks ? chosenAreas : {},
+    models,
+    ...(previous?.tools ? { tools: previous.tools } : {}),
+  };
 }
 
-keep(join(docs, 'index.md'), '# Index\n\nCatalog of the wiki: one line per page, `- [[file-name]]: summary`, grouped by type.\n');
-keep(join(docs, 'log.md'), `# Log\n\nAppend-only. Entry format: \`## [YYYY-MM-DD] kind | title\`.\n\n## [${today}] init | Superwiki vault created\n`);
-
-// The viewer snapshot and the local server's address are per-machine and regenerated on demand.
-write(join(dotdir, '.gitignore'), 'data.js\nserver.json\n');
-refresh(join(assets, 'sw.mjs'), join(dotdir, 'sw.mjs'));
-refresh(join(assets, 'viewer.html'), join(docs, 'viewer.html'));
-for (const t of ['page.md', ...(tasks ? ['task.md', 'plan.md', 'guide.md'] : [])]) refresh(join(assets, 'templates', t), join(dotdir, 'templates', t));
-
-const config = {
-  version: 1,
-  name: previous?.name ?? basename(root),
-  tasks,
-  areas: tasks ? (Object.keys(areas).length ? areas : previous?.areas ?? { T: 'Tasks' }) : {},
-  models: previous?.models ?? { plan: {}, implement: {} },
-  ...(previous?.tools ? { tools: previous.tools } : {}),
-};
-write(configPath, JSON.stringify(config, null, 2) + '\n');
-
-// Schema block: {{#tasks}}..{{/tasks}} kept with the task module, {{^tasks}}..{{/tasks}} without it.
-const block = readFileSync(join(assets, 'agents-block.md'), 'utf8')
-  .replace(/\{\{#tasks\}\}\n([\s\S]*?)\{\{\/tasks\}\}\n/g, (_, body) => (tasks ? body : ''))
-  .replace(/\{\{\^tasks\}\}\n([\s\S]*?)\{\{\/tasks\}\}\n/g, (_, body) => (tasks ? '' : body))
-  .trimEnd();
-const agentsPath = join(root, 'AGENTS.md');
-const managed = /<!-- sw:start[\s\S]*?<!-- sw:end -->/;
-if (!existsSync(agentsPath)) write(agentsPath, `# Agent instructions\n\n${block}\n`);
-else {
-  const text = readFileSync(agentsPath, 'utf8');
-  write(agentsPath, managed.test(text) ? text.replace(managed, () => block) : `${text.trimEnd()}\n\n${block}\n`, 'AGENTS.md (Superwiki block)');
+// The schema block keeps {{#tasks}}..{{/tasks}} parts with the task module and
+// {{^tasks}}..{{/tasks}} parts without it.
+function schemaBlock(tasks) {
+  return readFileSync(join(ASSETS, 'agents-block.md'), 'utf8')
+    .replace(/\{\{#tasks\}\}\n([\s\S]*?)\{\{\/tasks\}\}\n/g, (_, body) => (tasks ? body : ''))
+    .replace(/\{\{\^tasks\}\}\n([\s\S]*?)\{\{\/tasks\}\}\n/g, (_, body) => (tasks ? '' : body))
+    .trimEnd();
 }
 
-// Claude Code reads CLAUDE.md, not AGENTS.md; an import keeps one source of truth.
-const claudePath = join(root, 'CLAUDE.md');
-if (!existsSync(claudePath)) {
-  writeFileSync(claudePath, '@AGENTS.md\n');
-  report.push('created   CLAUDE.md (imports AGENTS.md)');
-} else if (!/AGENTS\.md/.test(readFileSync(claudePath, 'utf8'))) {
-  report.push('note      CLAUDE.md does not mention AGENTS.md; add a line "@AGENTS.md" so Claude Code reads the Superwiki rules');
-} else report.push('kept      CLAUDE.md');
-
-console.log(report.join('\n'));
-console.log(`\nvault: docs/  tasks: ${tasks ? `on (areas: ${Object.keys(config.areas).join(', ')})` : 'off'}`);
-if (foreign.length) console.log(`\ndocs/ already had content (${foreign.slice(0, 8).join(', ')}${foreign.length > 8 ? ', ...' : ''}). It was left untouched and is outside the vault; sw-migrate converts it.`);
+function writeVault(docs, tasks, report) {
+  const folders = ['raw/assets', 'wiki', ...(tasks ? ['tasks', 'plans'] : [])];
+  report.folder(docs);
+  for (const folder of ['raw', ...folders, '.sw', '.sw/templates']) report.folder(join(docs, folder));
+  // Git drops empty folders; the vault needs them to exist.
+  for (const folder of folders) {
+    if (!readdirSync(join(docs, folder)).length) writeFileSync(join(docs, folder, '.gitkeep'), '');
+  }
+
+  const today = new Date().toISOString().slice(0, 10);
+  report.keep(join(docs, 'index.md'), '# Index\n\nCatalog of the wiki: one line per page, `- [[file-name]]: summary`, grouped by type.\n');
+  report.keep(join(docs, 'log.md'), `# Log\n\nAppend-only. Entry format: \`## [YYYY-MM-DD] kind | title\`.\n\n## [${today}] init | Superwiki vault created\n`);
+
+  // The viewer snapshot and the local server's address are per-machine and regenerated on demand.
+  report.write(join(docs, '.sw', '.gitignore'), 'data.js\nserver.json\n');
+  report.copy(join(ASSETS, 'sw.mjs'), join(docs, '.sw', 'sw.mjs'));
+  report.copy(join(ASSETS, 'viewer.html'), join(docs, 'viewer.html'));
+  for (const template of ['page.md', ...(tasks ? ['task.md', 'plan.md', 'guide.md'] : [])]) {
+    report.copy(join(ASSETS, 'templates', template), join(docs, '.sw', 'templates', template));
+  }
+}
+
+function writeAgentRules(root, tasks, report) {
+  const block = schemaBlock(tasks);
+  const agentsPath = join(root, 'AGENTS.md');
+  if (existsSync(agentsPath)) {
+    const text = readFileSync(agentsPath, 'utf8');
+    const updated = MANAGED_BLOCK.test(text) ? text.replace(MANAGED_BLOCK, () => block) : `${text.trimEnd()}\n\n${block}\n`;
+    report.write(agentsPath, updated, 'AGENTS.md (Superwiki block)');
+  } else {
+    report.write(agentsPath, `# Agent instructions\n\n${block}\n`);
+  }
+
+  // Claude Code reads CLAUDE.md, not AGENTS.md; an import keeps one source of truth.
+  const claudePath = join(root, 'CLAUDE.md');
+  if (!existsSync(claudePath)) {
+    writeFileSync(claudePath, '@AGENTS.md\n');
+    report.line('created', 'CLAUDE.md (imports AGENTS.md)');
+  } else if (/AGENTS\.md/.test(readFileSync(claudePath, 'utf8'))) {
+    report.line('kept', 'CLAUDE.md');
+  } else {
+    report.line('note', 'CLAUDE.md does not mention AGENTS.md; add a line "@AGENTS.md" so Claude Code reads the Superwiki rules');
+  }
+}
+
+// What was in docs/ before the first init and is not part of a vault.
+function foreignEntries(docs) {
+  if (!existsSync(docs)) return [];
+  return readdirSync(docs).filter(name => !name.startsWith('.') && !VAULT_ENTRIES.includes(name));
+}
+
+function main(argv) {
+  const options = parseArgs(argv);
+  if (options.help) {
+    console.log(HELP);
+    return;
+  }
+  const root = resolve(options.root);
+  const docs = join(root, 'docs');
+  const configPath = join(docs, '.sw', 'config.json');
+  const previous = readConfig(configPath);
+  const tasks = options.tasks ?? previous?.tasks ?? null;
+  if (tasks === null) throw new UsageError('say --tasks or --no-tasks');
+
+  const foreign = previous ? [] : foreignEntries(docs);
+  const hadTaskFiles = !previous && existsSync(join(docs, 'tasks'));
+  const report = createReport(root);
+  writeVault(docs, tasks, report);
+  const config = buildConfig(previous, { root, tasks, areas: options.areas });
+  report.write(configPath, JSON.stringify(config, null, 2) + '\n');
+  writeAgentRules(root, tasks, report);
+
+  console.log(report.lines.join('\n'));
+  console.log(`\nvault: docs/  tasks: ${tasks ? `on (areas: ${Object.keys(config.areas).join(', ')})` : 'off'}`);
+  if (foreign.length) {
+    const shown = `${foreign.slice(0, 8).join(', ')}${foreign.length > 8 ? ', ...' : ''}`;
+    // After sw-migrate the task files are already there; only a first init on old docs needs the hint.
+    const hint = hadTaskFiles ? '' : ' If it holds a task index, sw-migrate converts it.';
+    console.log(`\ndocs/ already had content (${shown}). It was left untouched and is outside the vault.${hint}`);
+  }
+}
+
+try {
+  main(process.argv.slice(2));
+} catch (error) {
+  if (!(error instanceof UsageError)) throw error;
+  console.error(error.message);
+  process.exitCode = 2;
+}

package/skills/sw-migrate/SKILL.md

@@ -5,7 +5,7 @@ description: Use when the user wants to convert an existing docs folder, task in
 
 # sw-migrate
 
-Converts a project that tracks work in markdown tables into a Superwiki vault: one file per task, wikilinks, an append-only log. A script does the bulk conversion from a mapping you write; you only read samples, never the whole index.
+Converts a project that tracks work in markdown tables into a Superwiki vault: one file per task, wikilinks, an append-only log. A script inspects the existing files and does the conversion from a mapping you write. You never read the index yourself; it can be hundreds of kilobytes.
 
 Run everything from the project root. `<skill-dir>` is this skill's directory; `<init-dir>` is the sw-init skill's directory (a sibling folder).
 
@@ -19,43 +19,54 @@ Stop and tell the user if any of these fails; do not work around them.
 
 ## Steps
 
-1. **Branch**: `git switch -c sw-migrate`. All changes stay on this branch. Never merge, push or delete the branch yourself.
-2. **Sample, do not read.** The index can be hundreds of kilobytes.
-   - `ls docs docs/*/ | head -60` for the layout.
-   - `grep -n '^|' docs/<index> | awk 'length > 0' | cut -c1-200 | head -40` for table headers and a few rows.
-   - `grep -n '^## ' docs/<detail file> | head` and about 30 lines of one section, if tasks have detail files.
-   - The status values in use: `grep -o '| *[A-Za-z ✅]* *|' ... | sort | uniq -c` on the status column, or read ten rows.
-3. **Write the mapping** to a temporary file outside the repo. `node <skill-dir>/scripts/migrate.mjs --help` prints the format. Decide:
-   - which columns are id, title, status, dependencies, milestone, order, dates;
-   - every status value → `todo`, `in-progress`, `done` or `cancelled`;
+1. **Branch.** The conversion belongs on its own branch. If the project lets you run git, `git switch -c sw-migrate`; if it does not, or you are already on a branch the user made for this, ask the user to confirm the branch and stay on it. Never merge, push or delete a branch yourself.
+2. **Inspect**: `node <skill-dir>/scripts/migrate.mjs --inspect`. It prints, for every file that lists tasks: the tables with their line, row count and columns, the values of status-like columns with their counts, and how many per-task headings a file has. It also warns when `docs/wiki/`, `docs/tasks/` or `docs/plans/` already exist. This output is all you need for the mapping; do not open the files.
+3. **Write the mapping** to a temporary file outside the repo. `node <skill-dir>/scripts/migrate.mjs --help` prints the format. Decide, from the inspect output:
+   - the index file, and which columns are id, title, status, dependencies, milestone, order and dates;
+   - every status value → `todo`, `in-progress`, `done` or `cancelled`. Write the value as inspect shows it; decoration such as a check mark is already stripped;
    - the marker for soft dependencies, if the project has them;
-   - which other columns to keep as body sections (sources, notes);
+   - other columns worth keeping: as frontmatter (`fields`, for short values such as a review class) or as a body section (`sections`, for prose such as sources and notes);
    - where per-task detail sections live and their heading prefix;
-   - the changelog file and its columns, if any.
-4. **Dry run**: `node <skill-dir>/scripts/migrate.mjs --mapping <file> --dry-run`.
-5. **Show the user the mapping and the dry-run report, and wait for approval.** State the task counts per status next to the project's own numbers (its summary table, or a `grep -c`). If they differ, find out why before going on. Fix every line under `problems`.
-6. **Convert**: the same command without `--dry-run`, then the `init.mjs` command the report prints (replace the area names with real ones), then `node docs/.sw/sw.mjs status` and `node docs/.sw/sw.mjs lint`.
-7. **Verify**: `status` totals equal the dry-run counts and the project's own numbers; `lint` has 0 errors. Lint errors here are real inconsistencies in the source (a task started before its dependency finished, a dependency cycle). Report them; do not edit task files to make them pass.
+   - the table that is the changelog, if any. It may be in the index file itself;
+   - `archiveAlso`: anything in `docs/wiki/`, `docs/tasks/` or `docs/plans/` that the mapping does not consume. Superwiki owns those folders.
+4. **Dry run**: `node <skill-dir>/scripts/migrate.mjs --mapping <file> --dry-run`. The report has two parts:
+   - `PROBLEMS`: each one must be fixed, in the mapping or in the files, and the dry run repeated until it says `no problems`. An unknown status is a question for the user, not a guess.
+   - "For information": files whose links were rewritten, detail sections without a row, links that now point into the archive. Nothing to fix; pass the counts on.
+5. **Show the user the mapping and the dry-run report, and wait for approval.** Put the task counts per status next to the project's own numbers (inspect's status counts, or the project's summary table). If they differ, find out why before going on.
+6. **Convert**: the same command without `--dry-run`, then the `init.mjs` command the report prints, with real area names in place of the repeated ids, then `node docs/.sw/sw.mjs status` and `node docs/.sw/sw.mjs lint`.
+7. **Verify.** `status` shows `ready` and `blocked` where the report said `todo`: their sum must equal the todo count, and the other counts must match as they are. `lint` must have 0 errors. An error here is a real inconsistency in the source (a task started before its dependency finished, a dependency cycle): report it; do not edit task files to make it pass.
 8. **Report** what moved where, the counts, the lint result, and what is left for a human decision (next section). Commit only if the user asks.
 
-## What the script does not do
+## What the conversion does
+
+So that you can tell the user without looking:
+
+- every row of the task tables becomes `docs/tasks/<ID>.md`, with its detail section and the kept columns as the body;
+- the changelog becomes `docs/log.md`, oldest entry first;
+- the index, the detail files, the changelog and everything in `archiveAlso` move to `docs/legacy/`, keeping their layout;
+- links to tasks become wikilinks everywhere under `docs/`, so other documents are edited too; the report lists them;
+- `init.mjs` then adds `docs/index.md`, the viewer, the CLI and the Superwiki block in `AGENTS.md`.
+
+## What it leaves for the user
 
 Tell the user about each of these; act only on what they choose.
 
 | Left over | Where it is | Options |
-|---|---|---|
-| Rules and conventions written in the old index | `docs/legacy/` | most are replaced by the Superwiki block in `AGENTS.md`; project-specific ones go to `AGENTS.md` outside the Superwiki block |
+| --- | --- | --- |
+| Rules and conventions written in the old index | `docs/legacy/` | most are replaced by the Superwiki block in `AGENTS.md`; project-specific ones go to `AGENTS.md` outside that block |
 | Milestones, glossaries, decision records in the old index | `docs/legacy/` | turn into `docs/wiki/` pages with `type:` and `summary:`, and list them in `index.md` |
-| Research, specs, decisions in other folders | unchanged, outside the vault | leave, or move into `raw/` (sources) or `wiki/` (maintained pages) |
-| Plans written by other tools | unchanged | leave, or move to `docs/plans/<ID>-plan.md` when a plan belongs to exactly one task |
-| The old viewer or scripts that parse the old index | unchanged | delete once `docs/viewer.html` shows the same numbers |
-| Instructions in `AGENTS.md` / `CLAUDE.md` that describe the old index | unchanged | rewrite to point at the Superwiki rules; show the diff first |
+| Research, specs, decisions in other folders | in place, outside the vault | leave, or move into `raw/` (sources) or `wiki/` (maintained pages) |
+| Plans written by other tools | in place, or in the archive if they were under `docs/plans/` | leave, or turn one into `docs/plans/<ID>-plan.md` when it belongs to exactly one task |
+| The old viewer or scripts that parse the old index | in place | delete once `docs/viewer.html` shows the same numbers |
+| Instructions in `AGENTS.md` / `CLAUDE.md`, and links in files outside `docs/`, that point at the old index | in place | rewrite to point at the Superwiki rules and the new task files; show the diff first |
+| Task files that carry a long history (plan, review record, daily notes in one section) | `docs/tasks/` | leave; new tasks keep the plan in `docs/plans/` and history in the log |
 
 `docs/legacy/` is an archive, not part of the vault. Delete it only when the user says so.
 
 ## Common mistakes
 
-- Reading the whole index to "understand it first". Samples are enough; the dry run tells you what did not parse.
+- Opening the index or the detail files to "understand them first". Inspect and the dry run tell you what is there and what did not parse.
 - Mapping a status the script reported as unknown to `todo` without asking. Ask what it means.
+- Leaving a line under `PROBLEMS` and converting anyway.
 - Fixing lint errors by changing statuses. The old data was inconsistent; the user decides which side is right.
-- Hand-editing hundreds of links. If the script missed a link shape, fix the mapping or report it.
+- Hand-editing links. If the script missed a link shape, report it.