wormajs 1.0.0-beta.0 → 1.0.0-beta.1

package/dist/bin/actions.js

@@ -243,15 +243,35 @@ function createTable(head) {
         style: { head: [], border: [] },
     });
 }
-/** `worma diff` / `worma diff <id>` / `worma diff latest` */
-async function actionDiff(id, { list, project }) {
+/** `worma diff` / `worma diff <id>` / `worma diff latest` / `worma diff --remove <id>` */
+async function actionDiff(id, { list, remove, project }) {
     const projectPath = project
         ? (node_path_1.default.isAbsolute(project) ? project : node_path_1.default.resolve(process.cwd(), project))
         : process.cwd();
     // Mirror `actionGen`: always resolve the cache from CWD so monorepo
     // sub-packages share one unified cache root.
     (0, config_1.setGlobalConfig)({ cacheRoot: process.cwd() });
-    const { countChanges, getChange, listChanges } = await Promise.resolve().then(() => __importStar(require('../functions/changeReport')));
+    const { countChanges, getChange, listChanges, removeChange } = await Promise.resolve().then(() => __importStar(require('../functions/changeReport')));
+    // `worma diff --remove 0007` (also `worma diff 0007 --remove` / `--remove latest`)
+    if (remove) {
+        // `--remove` accepts the id itself; fall back to the positional argument so
+        // both `--remove 0007` and `0007 --remove` work.
+        const targetId = typeof remove === 'string' ? remove : id;
+        if (!targetId) {
+            console.log(`\n  ${theme_1.theme.warning('?')} Nothing to remove: pass a change id, e.g. ${theme_1.theme.label('worma diff --remove 0007')}.\n`);
+            process.exitCode = 1;
+            return;
+        }
+        const removedId = await removeChange(projectPath, targetId);
+        if (!removedId) {
+            console.log(`\n  ${theme_1.theme.warning('?')} No change record found for "${targetId}".\n`);
+            process.exitCode = 1;
+            return;
+        }
+        console.log(`\n  ${theme_1.theme.success('✔')} Removed change record ${theme_1.theme.label(removedId)}.`);
+        console.log(`  ${theme_1.theme.dim('Run `worma diff` to list the remaining records.')}\n`);
+        return;
+    }
     if (!id || list) {
         const summaries = await listChanges(projectPath);
         if (summaries.length === 0) {

package/dist/bin/cli.js

@@ -27,6 +27,7 @@ program
     .description('browse recorded API changes')
     .argument('[id]', 'change id (e.g. 0007) or `latest`; omit to list all')
     .option('-l, --list', 'list all recorded changes')
+    .option('-r, --remove [id]', 'delete a recorded change (e.g. `worma diff --remove 0007`)')
     .option('-p, --project <path>', 'project directory')
     .action(actions_1.actionDiff);
 program.parse(process.argv);

package/dist/functions/changeReport.js

@@ -9,6 +9,7 @@ exports.changesDirPath = changesDirPath;
 exports.captureChange = captureChange;
 exports.listChanges = listChanges;
 exports.getChange = getChange;
+exports.removeChange = removeChange;
 const promises_1 = __importDefault(require("node:fs/promises"));
 const node_path_1 = __importDefault(require("node:path"));
 const config_1 = require("../config");
@@ -218,6 +219,40 @@ async function getChange(projectPath, id) {
     const record = await readRecord(projectPath, resolvedId);
     return record ?? undefined;
 }
+/** Ids are always the zero-padded sequence written by {@link captureChange}. */
+const CHANGE_ID_RE = /^\d+$/;
+/**
+ * Delete a single recorded change.
+ *
+ * @param projectPath absolute path of the project root
+ * @param id `"0007"` or the alias `"latest"` (newest record)
+ *
+ * `index.json#changeSeq` is deliberately left untouched: it only ever allocates
+ * new* ids, so keeping it monotonic guarantees the deleted id is never handed
+ * out again for a different record.
+ *
+ * @returns the id that was deleted, or `undefined` when nothing matched
+ */
+async function removeChange(projectPath, id) {
+    let resolvedId = id;
+    if (!id || id === exports.LATEST_CHANGE_ID) {
+        const latestId = await resolveLatestId(projectPath);
+        if (!latestId)
+            return undefined;
+        resolvedId = latestId;
+    }
+    // Guards against deleting anything outside the changes directory
+    // (`--remove ../../some-file` would otherwise unlink an arbitrary file).
+    if (!CHANGE_ID_RE.test(resolvedId))
+        return undefined;
+    try {
+        await promises_1.default.unlink(recordFile(projectPath, resolvedId));
+        return resolvedId;
+    }
+    catch {
+        return undefined;
+    }
+}
 /**
  * Return the newest change-record id.
  *

package/dist/helper/template/index.js

@@ -387,7 +387,20 @@ class TemplateHelper {
         // --- Phase 1: Per-tag streaming (render → collect → batch write) ---
         const nonDirTagTpls = tpls.filter(f => !f.insideTagDir && f.templateType === 'tag');
         const dirTpls = tpls.filter(f => f.insideTagDir);
-        const effectiveTags = changedTags ? tags.filter(t => changedTags.has(t)) : tags;
+        // Incremental rendering normally skips a tag whose hash is unchanged, which
+        // assumes its previously generated files are still on disk. If those files
+        // were deleted manually while the cache survived, the unchanged tag would
+        // never be emitted again. So an unchanged tag is also re-rendered whenever
+        // any of its expected artifacts is missing from the output directory.
+        let effectiveTags;
+        if (changedTags) {
+            const unchangedTags = tags.filter(t => !changedTags.has(t));
+            const missingTags = await this.collectTagsWithMissingArtifacts(unchangedTags, nonDirTagTpls, dirTpls, tagApisMap, outputDir);
+            effectiveTags = tags.filter(t => changedTags.has(t) || missingTags.has(t));
+        }
+        else {
+            effectiveTags = tags;
+        }
         let tagFilesWritten = 0;
         logger_1.logger.debug('Phase 1: Per-tag streaming', {
             totalTags: tags.length,
@@ -461,6 +474,45 @@ class TemplateHelper {
         logger_1.logger.debug('Generation summary', { totalOutputFiles: allFilePaths.length });
         return { filePaths: allFilePaths };
     }
+    /**
+     * Detect tags whose previously generated artifacts are missing from disk.
+     *
+     * Mirrors the path derivation used by `renderOne` / `expandByApi` to compute
+     * the expected output path of every per-tag file without rendering anything,
+     * then returns the tags that must be re-rendered because at least one of
+     * their files is gone.
+     */
+    async collectTagsWithMissingArtifacts(candidateTags, nonDirTagTpls, dirTpls, tagApisMap, outputDir) {
+        const missingTags = new Set();
+        for (const tag of candidateTags) {
+            const tagApis = tagApisMap.get(tag);
+            const expectedPaths = [];
+            // Flat `[tag]`-named templates, e.g. `services/[tag].ts.handlebars`
+            for (const tf of nonDirTagTpls) {
+                expectedPaths.push(stripExt(normalizeSlashes(tf.relativePath.replace(constant_1.TemplatePlaceholder.TAG, tag))));
+            }
+            // Tag-dir templates, e.g. `[tag]/index.ts` and `[tag]/[api].ts`
+            for (const tf of dirTpls) {
+                if (tf.templateType === 'api') {
+                    for (const api of tagApis?.apis || []) {
+                        expectedPaths.push(stripExt(normalizeSlashes(tf.relativePath
+                            .replace(constant_1.TemplatePlaceholder.TAG, tag)
+                            .replace(constant_1.TemplatePlaceholder.API, api.name))));
+                    }
+                }
+                else {
+                    expectedPaths.push(stripExt(normalizeSlashes(tf.relativePath.replace(constant_1.TemplatePlaceholder.TAG, tag))));
+                }
+            }
+            for (const relPath of expectedPaths) {
+                if (!(await (0, utils_1.existsPromise)(node_path_1.default.join(outputDir, relPath)))) {
+                    missingTags.add(tag);
+                    break;
+                }
+            }
+        }
+        return missingTags;
+    }
     /**
      * Remove generated artifacts of tags that no longer exist in the spec.
      *

package/dist/index.js

@@ -17,7 +17,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.resolveWorkspaces = exports.logger = exports.generate = exports.writeSourceSnapshot = exports.sourceDocumentHash = exports.readSourceSnapshot = exports.diffSourceDocument = exports.diffApis = exports.apiDiffKey = exports.listChanges = exports.LATEST_CHANGE_ID = exports.getChange = exports.countChanges = exports.defineConfig = exports.createConfig = exports.setGlobalConfig = exports.checkUpdates = void 0;
+exports.resolveWorkspaces = exports.logger = exports.generate = exports.writeSourceSnapshot = exports.sourceDocumentHash = exports.readSourceSnapshot = exports.diffSourceDocument = exports.diffApis = exports.apiDiffKey = exports.removeChange = exports.listChanges = exports.LATEST_CHANGE_ID = exports.getChange = exports.countChanges = exports.defineConfig = exports.createConfig = exports.setGlobalConfig = exports.checkUpdates = void 0;
 var checkUpdates_1 = require("./checkUpdates");
 Object.defineProperty(exports, "checkUpdates", { enumerable: true, get: function () { return checkUpdates_1.checkUpdates; } });
 var config_1 = require("./config");
@@ -31,6 +31,7 @@ Object.defineProperty(exports, "countChanges", { enumerable: true, get: function
 Object.defineProperty(exports, "getChange", { enumerable: true, get: function () { return changeReport_1.getChange; } });
 Object.defineProperty(exports, "LATEST_CHANGE_ID", { enumerable: true, get: function () { return changeReport_1.LATEST_CHANGE_ID; } });
 Object.defineProperty(exports, "listChanges", { enumerable: true, get: function () { return changeReport_1.listChanges; } });
+Object.defineProperty(exports, "removeChange", { enumerable: true, get: function () { return changeReport_1.removeChange; } });
 var diffApis_1 = require("./functions/diffApis");
 Object.defineProperty(exports, "apiDiffKey", { enumerable: true, get: function () { return diffApis_1.apiDiffKey; } });
 Object.defineProperty(exports, "diffApis", { enumerable: true, get: function () { return diffApis_1.diffApis; } });

package/dist/plugins/presets/aiDoc.js

@@ -73,8 +73,15 @@ function aiDoc(config) {
             });
             if (agentValue) {
                 const agentsToInstall = resolveInstallAgents(agentValue);
-                for (const agent of agentsToInstall) {
-                    installSkill(aidocsDir, agent, projectPath);
+                if (agentsToInstall.length) {
+                    for (const agent of agentsToInstall) {
+                        installSkill(aidocsDir, agent, projectPath);
+                    }
+                    // The skill now lives in each agent's own skills directory. Keeping the
+                    // generated copy under the output directory would store the same files
+                    // twice, so it is dropped — but only after *every* install succeeded:
+                    // a failing install throws above and leaves the source in place.
+                    removeSkillSourceDir(aidocsDir);
                 }
             }
         },
@@ -147,6 +154,24 @@ function installSkill(skillPath, agent, projectPath) {
         throw logger_1.logger.throwError(error);
     }
 }
+/**
+ * Delete the generated skill directory once it has been installed.
+ *
+ * `skills add` copies the whole directory into the agent's own skills folder,
+ * so the copy under the generator output is pure duplication.
+ *
+ * A removal failure is reported but never thrown: the install already
+ * succeeded, and failing the whole generation over a leftover directory would
+ * be worse than keeping it.
+ */
+function removeSkillSourceDir(skillPath) {
+    try {
+        node_fs_1.default.rmSync(skillPath, { recursive: true, force: true });
+    }
+    catch (error) {
+        console.error(`${prefix}Failed to remove the generated skill directory "${skillPath}".`, error?.stack ?? error);
+    }
+}
 /**
  * Parse a `key=value` configuration file (same format as an environment file).
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wormajs",
-  "version": "1.0.0-beta.0",
+  "version": "1.0.0-beta.1",
   "description": "A modern OpenAPI code generator - Generate type-safe API clients from OpenAPI specs",
   "author": "worma",
   "license": "MIT",

package/typings/index.d.ts

@@ -658,6 +658,19 @@ export declare function listChanges(projectPath: string): Promise<ChangeSummary[
  * @param id `"0007"` or the alias `"latest"` (newest record)
  */
 export declare function getChange(projectPath: string, id: string): Promise<Change | undefined>;
+/**
+ * Delete a single recorded change.
+ *
+ * @param projectPath absolute path of the project root
+ * @param id `"0007"` or the alias `"latest"` (newest record)
+ *
+ * `index.json#changeSeq` is deliberately left untouched: it only ever allocates
+ * new* ids, so keeping it monotonic guarantees the deleted id is never handed
+ * out again for a different record.
+ *
+ * @returns the id that was deleted, or `undefined` when nothing matched
+ */
+export declare function removeChange(projectPath: string, id: string): Promise<string | undefined>;
 /** A newly added or removed API (identified by `method` + `path`). */
 export interface ApiChange {
 	method: string;