@rse/ase 2.0.2 → 2.0.4

package/dst/ase-task-board-web-server.js

@@ -192,7 +192,7 @@ const attachmentDocument = async (log, id, n) => {
         try {
             text = new TextDecoder("utf-8", { fatal: true }).decode(a.content);
         }
-        catch (_err) {
+        catch {
             text = "\0";
         }
         if (text.includes("\0"))
@@ -364,7 +364,7 @@ const registerPageRoutes = (server, log) => {
 };
 /*  the filter query of a request (its optional "filter" query parameter)  */
 const filterQuery = (request) => typeof request.query.filter === "string" ? request.query.filter : "";
-/*  register the view routes of the web board: the board, the graph, the statistics, and the tasks with their attachments  */
+/*  register the view routes of the web board: the board, the task and agent graphs, the statistics, the new task text, and the tasks with their attachments  */
 const registerViewRoutes = (server, log) => {
     /*  the board model, reduced onto the tasks matching the filter query  */
     server.route({

package/dst/ase-task-board.js

@@ -78,7 +78,7 @@ export default class TaskBoardCommand {
                 "(restart it with \"ase service stop\" once no agent session uses it)");
         return port;
     }
-    /*  open an URL in the default browser of the platform, waiting only for
+    /*  open a URL in the default browser of the platform, waiting only for
         the opener to spawn, as e.g. "xdg-open" can stay attached to the browser  */
     async openBrowser(url) {
         const [cmd, ...args] = process.platform === "darwin" ? ["open", url] :

package/dst/ase-task-format.js

@@ -745,7 +745,7 @@ export const bindEffort = (plan) => {
     if (effort !== null && effort.hash === "")
         plan.header.Effort = `${effort.size} ${hash}`;
     else if (effort === null || effort.hash !== hash)
-        Reflect.deleteProperty(plan.header, "Effort");
+        delete plan.header.Effort;
     return plan;
 };
 /*  derive the title of a task plan from its body as defined by the

package/dst/ase-task-store-client.js

@@ -58,6 +58,7 @@ export class LocalTaskStoreClient {
         this.configure("project.task.idscheme", spec);
         this.idscheme = (await this.core.projectSet(this.prjId, { idscheme: spec })).project.idscheme;
     }
+    /*  run an operation, yielding a fallback for a missing (404) task plan  */
     async missing(op, fallback) {
         try {
             return await op();
@@ -68,6 +69,7 @@ export class LocalTaskStoreClient {
             throw err;
         }
     }
+    /*  open the storage delegate shared by all clients of the store and register the project  */
     async open() {
         let entry = LocalTaskStoreClient.shared.get(this.storage.key);
         if (entry === undefined) {
@@ -99,6 +101,7 @@ export class LocalTaskStoreClient {
             throw err;
         }
     }
+    /*  release the shared storage delegate, closing it with its last client  */
     async close() {
         const entry = this.entry;
         if (entry === undefined)
@@ -107,7 +110,9 @@ export class LocalTaskStoreClient {
         if (--entry.refs === 0) {
             if (LocalTaskStoreClient.shared.get(this.storage.key) === entry)
                 LocalTaskStoreClient.shared.delete(this.storage.key);
-            await entry.opened.then((opened) => opened.store.close(), () => { });
+            await entry.opened.then((opened) => opened.store.close(), () => {
+                /*  a failed opening was already raised to its opener  */
+            });
         }
     }
     list(fields) {
@@ -153,11 +158,13 @@ export class LocalTaskStoreClient {
                 onChange();
         };
         let stopped = false;
+        let started = false;
         const start = async () => {
             await this.open();
             if (stopped)
                 await this.close();
             else {
+                started = true;
                 this.entry?.listeners.add(listener);
                 onState?.(true);
             }
@@ -167,6 +174,8 @@ export class LocalTaskStoreClient {
         });
         return () => {
             stopped = true;
+            if (!started)
+                return;
             this.entry?.listeners.delete(listener);
             this.close().catch(() => { });
         };
@@ -174,9 +183,10 @@ export class LocalTaskStoreClient {
 }
 /*  the interval of the keepalive pings of the event subscription (where a connection without
     any traffic for this interval is pinged and without any traffic for twice this interval
-    is terminated) and the timeout of its WebSocket handshake  */
+    is terminated), the timeout of its WebSocket handshake, and the timeout of the REST API requests  */
 const KEEPALIVE_MS = 15 * 1000;
 const HANDSHAKE_MS = 10 * 1000;
+const REQUEST_MS = 10 * 1000;
 /*  the remote client: the REST API of a task store server, with the
     project registered under its configured lifecycle model and task id
     scheme on first use only (afterwards adopting the lifecycle model and
@@ -231,7 +241,7 @@ export class RemoteTaskStoreClient {
             },
             body: body !== undefined ? JSON.stringify(body) : undefined,
             dispatcher: this.dispatcher,
-            signal: AbortSignal.timeout(10000),
+            signal: AbortSignal.timeout(REQUEST_MS),
             ignoreResponseError: true
         }).catch((err) => this.unreachable(err));
         if (r.status === 404 || tolerated.includes(r.status))
@@ -402,7 +412,7 @@ export class RemoteTaskStoreClient {
             headers: { Authorization: `Bearer ${this.token}` },
             responseType: "arrayBuffer",
             dispatcher: this.dispatcher,
-            signal: AbortSignal.timeout(10000),
+            signal: AbortSignal.timeout(REQUEST_MS),
             ignoreResponseError: true
         }).catch((err) => this.unreachable(err));
         if (r.status === 404)
@@ -489,7 +499,9 @@ export class RemoteTaskStoreClient {
             if (timer !== null)
                 clearTimeout(timer);
             ws?.close();
-            this.close().catch(() => { });
+            this.close().catch(() => {
+                /*  best-effort release of the dispatcher on unsubscribe  */
+            });
         };
     }
 }

package/dst/ase-task-store-core.js

@@ -143,7 +143,7 @@ export class TaskStoreCore {
         if (effort !== undefined)
             header.Effort = effort;
         else
-            Reflect.deleteProperty(header, "Effort");
+            delete header.Effort;
         return header;
     }
     /*  load an existing task plan of a registered project  */
@@ -408,7 +408,7 @@ export class TaskStoreCore {
                 if (lifecycle.states.includes(status) && !states.includes(status))
                     continue;
                 const att = fields === "usage" ? (await this.store.taskLoad(prjId, entry.id))?.attachment
-                    .find((att) => TaskFormat.isUsage(att) && att.Data !== undefined) : undefined;
+                    .find((a) => TaskFormat.isUsage(a) && a.Data !== undefined) : undefined;
                 const header = fields === "header" ? await this.boundHeader(prjId, entry) : entry.header;
                 result.push({
                     id: entry.id,
@@ -605,12 +605,10 @@ export class TaskStoreCore {
                     references: referrers.length
                 };
             }
-            else {
-                await this.store.taskSave(prjId, taskId, plan);
-                this.emitUpdated(prjId, taskId, plan, lifecycle, ["header"]);
-            }
+            await this.store.taskSave(prjId, taskId, plan);
+            this.emitUpdated(prjId, taskId, plan, lifecycle, ["header"]);
             return {
-                id: newId ?? taskId,
+                id: taskId,
                 status: TaskFormat.taskStatus(plan.header, lifecycle),
                 ...(status !== null ? { from } : {})
             };

package/dst/ase-task-store-issues.js

@@ -235,6 +235,19 @@ export const attachmentSync = (current, wanted) => {
     }
     return ops;
 };
+/*  reuse the cached promise of a load, else start the load and cache its promise
+    (dropped again on failure), so repeated and concurrent loads share one request  */
+export const memoize = (cache, key, load) => {
+    let promise = cache.get(key);
+    if (promise === undefined) {
+        promise = load();
+        cache.set(key, promise);
+        promise.catch(() => {
+            cache.delete(key);
+        });
+    }
+    return promise;
+};
 /*  the common base of the issue tracker storage plugins: the projects mapped onto
     their locations, the storage lifecycle polling for changes while opened and
     observed (without keeping the process alive), and the project listing  */
@@ -261,7 +274,9 @@ export class IssueTaskStoragePlugin {
         this.opened = this.now();
         if (this.poll > 0 && this.listener !== null) {
             this.timer = setInterval(() => {
-                this.pollAll().catch(() => { });
+                this.pollAll().catch(() => {
+                    /*  failures are already logged per project  */
+                });
             }, this.poll * 1000);
             this.timer.unref();
         }

package/dst/ase-task-store-plugin-ase.js

@@ -143,7 +143,7 @@ class FileTaskStoragePlugin {
             doc = parseYAML(text);
         }
         catch (err) {
-            this.ctx.log("warning", `malformed project file "${file}" ignored: ${err.message}`);
+            this.ctx.log("warning", `malformed project file "${file}" ignored: ${err instanceof Error ? err.message : String(err)}`);
             return { lifecycle: this.lifecycle, idscheme: this.idscheme, seqmark: 0, broken: true };
         }
         return {
@@ -339,7 +339,9 @@ class FileTaskStoragePlugin {
                 if (this.watcher !== w)
                     return;
                 this.watcher = null;
-                w.close().catch(() => { }).finally(() => {
+                w.close().catch(() => {
+                    /*  re-arm even if the outdated watcher fails to close  */
+                }).finally(() => {
                     this.arm();
                     this.schedule();
                 });
@@ -418,7 +420,9 @@ class FileTaskStoragePlugin {
             clearTimeout(this.timer);
         this.timer = setTimeout(() => {
             this.timer = null;
-            this.rescan().catch(() => { });
+            this.rescan().catch(() => {
+                /*  failures are already logged per directory  */
+            });
         }, 100);
         this.timer.unref();
     }

package/dst/ase-task-store-plugin-gitea.js

@@ -248,7 +248,7 @@ class GiteaTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
             await found(this.call(loc, this.gt.repos.issueDeleteLabel(loc.owner, loc.repo, label.id)).then(() => true)) : null;
         this.labels.delete(`${loc.owner}/${loc.repo}`);
         this.registry.delete(prjId);
-        return label !== undefined && result !== null;
+        return result !== null;
     }
     /*  ==== issue mapping ====  */
     /*  fetch the live issue of an issue number (null if none)  */
@@ -264,15 +264,8 @@ class GiteaTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         const key = `${loc.owner}/${loc.repo}#${issue.number}@${issue.updated_at}`;
         if (fresh)
             this.depends.delete(key);
-        let depends = this.depends.get(key);
-        if (depends === undefined) {
-            depends = found(this.all(loc, (query) => this.gt.repos.issueListIssueDependencies(loc.owner, loc.repo, String(issue.number), query)))
-                .then((issues) => (issues ?? []));
-            this.depends.set(key, depends);
-            depends.catch(() => {
-                this.depends.delete(key);
-            });
-        }
+        const depends = TaskIssues.memoize(this.depends, key, () => found(this.all(loc, (query) => this.gt.repos.issueListIssueDependencies(loc.owner, loc.repo, String(issue.number), query)))
+            .then((issues) => (issues ?? [])));
         return (await depends).filter((blocker) => inRepo(loc, blocker));
     }
     /*  the comments of an issue  */
@@ -323,10 +316,10 @@ class GiteaTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
             if (label === undefined) {
                 label = await this.call(loc, this.gt.repos.issueCreateLabel(loc.owner, loc.repo, { name, color: name.startsWith("ase:") ? "#c5def5" : "#ededed" })).catch(async (err) => {
                     /*  tolerate a label created concurrently  */
-                    const label = statusOf(err) === 422 ? (await this.labelMap(loc, true)).get(name.toLowerCase()) : undefined;
-                    if (label === undefined)
+                    const existing = statusOf(err) === 422 ? (await this.labelMap(loc, true)).get(name.toLowerCase()) : undefined;
+                    if (existing === undefined)
                         throw err;
-                    return label;
+                    return existing;
                 });
                 const map = await this.labelMap(loc);
                 map.set(name.toLowerCase(), label);
@@ -383,15 +376,8 @@ class GiteaTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         if (typeof assignee === "string" && assignee !== "") {
             const logins = issue?.assignees?.map((user) => user.login) ?? [];
             const key = `${loc.owner}/${loc.repo}`;
-            let users = this.assignable.get(key);
-            if (users === undefined) {
-                users = this.call(loc, this.gt.repos.repoGetAssignees(loc.owner, loc.repo))
-                    .then((list) => new Set(list.map((user) => (user.login ?? "").toLowerCase())));
-                this.assignable.set(key, users);
-                users.catch(() => {
-                    this.assignable.delete(key);
-                });
-            }
+            const users = TaskIssues.memoize(this.assignable, key, () => this.call(loc, this.gt.repos.repoGetAssignees(loc.owner, loc.repo))
+                .then((list) => new Set(list.map((user) => (user.login ?? "").toLowerCase()))));
             if (logins.includes(assignee))
                 assignees = logins;
             else if ((await users).has(assignee.toLowerCase()))
@@ -447,11 +433,7 @@ class GiteaTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         own write or detected change of the issue), as e.g. the boards load the plan
         and its attachments in separate operations  */
     load(prjId, loc, n) {
-        const key = `${prjId}#${n}`;
-        const cached = this.loads.get(key);
-        if (cached !== undefined)
-            return cached;
-        const load = (async () => {
+        return TaskIssues.memoize(this.loads, `${prjId}#${n}`, async () => {
             const [issue, comments] = await Promise.all([
                 this.fetch(loc, n),
                 n === 0 ? [] : found(this.comments(loc, n))
@@ -459,12 +441,7 @@ class GiteaTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
             if (issue === null)
                 return null;
             return { issue, after: await this.blockers(loc, issue), comments: comments ?? [] };
-        })();
-        this.loads.set(key, load);
-        load.catch(() => {
-            this.loads.delete(key);
         });
-        return load;
     }
     /*  ==== task plans ====  */
     async taskList(prjId) {

package/dst/ase-task-store-plugin-github.js

@@ -73,7 +73,7 @@ class GitHubTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
     constructor(ctx) {
         super(ctx, "repositories", now);
         const options = ctx.options;
-        const token = options.token ?? process.env.GITHUB_TOKEN ?? process.env.GH_TOKEN ?? "";
+        const token = options.token ?? (process.env.GITHUB_TOKEN || process.env.GH_TOKEN) ?? "";
         if (typeof token !== "string" || token === "")
             throw new Error("task store: plugin \"github\" requires the \"token\" option (or $GITHUB_TOKEN resp. $GH_TOKEN)");
         const url = options.url ?? "https://github.com";
@@ -122,7 +122,7 @@ class GitHubTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         /*  explain a denied access (but not an exceeded rate limit), as the
             GitHub message names neither the token source nor the missing permission  */
         const source = options.token !== undefined ? "the \"token\" option (resp. \"project.task.store.token\")" :
-            process.env.GITHUB_TOKEN !== undefined ? "$GITHUB_TOKEN" : "$GH_TOKEN";
+            process.env.GITHUB_TOKEN !== undefined && process.env.GITHUB_TOKEN !== "" ? "$GITHUB_TOKEN" : "$GH_TOKEN";
         this.gh.hook.error("request", (err, request) => {
             const status = statusOf(err);
             if ((status === 401 || status === 403) && !/rate limit/i.test(err.message)) {
@@ -462,11 +462,7 @@ class GitHubTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         own write or detected change of the issue), as e.g. the boards load the plan
         and its attachments in separate operations  */
     load(prjId, loc, n) {
-        const key = `${prjId}#${n}`;
-        const cached = this.loads.get(key);
-        if (cached !== undefined)
-            return cached;
-        const load = (async () => {
+        return TaskIssues.memoize(this.loads, `${prjId}#${n}`, async () => {
             const [issue, comments] = await Promise.all([
                 this.fetch(loc, n),
                 n === 0 ? [] : found(this.gh.paginate(this.gh.rest.issues.listComments, { ...loc, issue_number: n, per_page: 100 }))
@@ -474,12 +470,7 @@ class GitHubTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
             if (issue === null)
                 return null;
             return { issue, after: await this.blockers(loc, issue), comments: comments ?? [] };
-        })();
-        this.loads.set(key, load);
-        load.catch(() => {
-            this.loads.delete(key);
         });
-        return load;
     }
     /*  ==== task plans ====  */
     async taskList(prjId) {

package/dst/ase-task-store-plugin-gitlab.js

@@ -240,14 +240,7 @@ class GitLabTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         const key = `${path}#${issue.iid}@${issue.updated_at}`;
         if (fresh)
             this.linked.delete(key);
-        let links = this.linked.get(key);
-        if (links === undefined) {
-            links = this.links.all(path, issue.iid).then((data) => data);
-            this.linked.set(key, links);
-            links.catch(() => {
-                this.linked.delete(key);
-            });
-        }
+        const links = TaskIssues.memoize(this.linked, key, () => this.links.all(path, issue.iid).then((data) => data));
         return (await links).filter((link) => link.link_type === "is_blocked_by" && link.project_id === issue.project_id);
     }
     /*  the (non-system) notes of an issue  */
@@ -451,11 +444,7 @@ class GitLabTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
         own write or detected change of the issue), as e.g. the boards load the plan
         and its attachments in separate operations  */
     load(prjId, path, n) {
-        const key = `${prjId}#${n}`;
-        const cached = this.loads.get(key);
-        if (cached !== undefined)
-            return cached;
-        const load = (async () => {
+        return TaskIssues.memoize(this.loads, `${prjId}#${n}`, async () => {
             const [issue, notes] = await Promise.all([
                 this.fetch(path, n),
                 n === 0 ? [] : found(this.comments(path, n))
@@ -463,12 +452,7 @@ class GitLabTaskStoragePlugin extends TaskIssues.IssueTaskStoragePlugin {
             if (issue === null)
                 return null;
             return { issue, after: await this.blockers(path, issue), notes: notes ?? [] };
-        })();
-        this.loads.set(key, load);
-        load.catch(() => {
-            this.loads.delete(key);
         });
-        return load;
     }
     /*  ==== task plans ====  */
     async taskList(prjId) {

package/dst/ase-task.js

@@ -341,7 +341,7 @@ export class Task {
             try {
                 url = new URL(store.value);
             }
-            catch (_err) {
+            catch {
                 throw unsupported();
             }
             if (url.hostname === "" || url.port === "" || url.username !== "" || url.password !== ""
@@ -444,21 +444,12 @@ export class Task {
             tag: Core.taskTag(plan)
         };
     }
-    /*  save a task as text under the given id; throws if its "Status:"
-        frontmatter key is unknown to the task lifecycle model or not
-        reachable from the previous status, on a conditional save (with
-        an entity tag) a problem 412 if the task was changed in the meantime,
-        and on a create-only save an error if the task already exists;
-        returns a warning (else empty) if the id does not conform to the task id scheme,
-        the entity tag of the just-saved plan, and the text of the just-saved plan
-        (as sent, re-serialized from its parsed form, i.e., before any
-        normalization by the task store, and with its elided attachment
-        payloads still elided, while they are restored from the stored
-        plan for the save itself); the entity tag
-        is the one the task store reports for the plan as stored, and only on a
-        task store server predating this report the one of the plan as sent (which
-        never is the one of a plan saved by somebody else in the meantime, so a
-        conditional save based on it at worst is refused spuriously)  */
+    /*  save a task as text under the given id, throwing on a "Status:" unknown to or not
+        reachable in the task lifecycle model, on a conditional save (with an entity tag) a
+        problem 412 on a concurrent change, and on a create-only save an error if the task
+        exists; elided attachment payloads are restored from the stored plan; returns the
+        id scheme warning (else empty), the entity tag of the saved plan (as reported by the
+        task store, else of the plan as sent), and the text of the plan as sent  */
     static async save(log, id, text, tag, create = false) {
         if (typeof text !== "string")
             throw new Error("task: text must be a string");
@@ -993,6 +984,17 @@ export default class TaskCommand {
                 process.exit(0);
             }
         });
+        /*  register the further CLI sub-commands  */
+        this.registerRequestCommands(task);
+        this.registerPlanCommands(task);
+        this.registerProjectCommands(task);
+        /*  register CLI sub-command group "ase task store"  */
+        new TaskStoreCommand(this.log).register(task);
+        /*  provide "ase task" command for further sub-commands  */
+        return task;
+    }
+    /*  register the CLI sub-commands requesting and assigning a task  */
+    registerRequestCommands(task) {
         /*  register CLI sub-command "ase task request"  */
         task
             .command("request")
@@ -1049,6 +1051,9 @@ export default class TaskCommand {
             }
             process.exit(0);
         });
+    }
+    /*  register the CLI sub-commands loading, viewing, editing, saving, deleting, and renaming a task  */
+    registerPlanCommands(task) {
         /*  register CLI sub-command "ase task load"  */
         task
             .command("load")
@@ -1130,6 +1135,9 @@ export default class TaskCommand {
                 this.log.write("info", `task: no task "${oldId}" to rename`);
             process.exit(result !== null ? 0 : 1);
         });
+    }
+    /*  register the CLI sub-commands of the task ids, the purging, and the project settings  */
+    registerProjectCommands(task) {
         /*  register CLI sub-command "ase task newid"  */
         task
             .command("newid")
@@ -1197,10 +1205,6 @@ export default class TaskCommand {
                 process.exit(0);
             }
         });
-        /*  register CLI sub-command group "ase task store"  */
-        new TaskStoreCommand(this.log).register(task);
-        /*  provide "ase task" command for further sub-commands  */
-        return task;
     }
 }
 /*  render a caught error as an MCP tool error result  */
@@ -1264,9 +1268,8 @@ export class TaskMCP {
             try {
                 const verbose = args.verbose ?? false;
                 const items = await Task.list(this.log);
-                const tasks = verbose ?
-                    items.map((item) => ({ id: item.id, status: item.status, title: item.title, mtime: item.mtime, effort: item.effort })) :
-                    items.map((item) => ({ id: item.id, status: item.status, title: item.title }));
+                const tasks = verbose ? items :
+                    items.map(({ id, status, title }) => ({ id, status, title }));
                 const result = { tasks };
                 return {
                     structuredContent: result,
@@ -1277,6 +1280,16 @@ export class TaskMCP {
                 return mcpError(err);
             }
         });
+        /*  register the further MCP tools  */
+        this.registerLoadTool(mcp);
+        this.registerSaveTool(mcp);
+        this.registerPlanTools(mcp);
+        this.registerStatusTools(mcp);
+        this.registerAssignTool(mcp);
+        this.registerIdTools(mcp);
+    }
+    /*  register the MCP tool loading a task plan  */
+    registerLoadTool(mcp) {
         /*  task load  */
         mcp.registerTool("ase_task_load", {
             title: "ASE task load",
@@ -1332,6 +1345,9 @@ export class TaskMCP {
                 return mcpError(err);
             }
         });
+    }
+    /*  register the MCP tool saving a task plan  */
+    registerSaveTool(mcp) {
         /*  task save  */
         mcp.registerTool("ase_task_save", {
             title: "ASE task save",
@@ -1408,6 +1424,9 @@ export class TaskMCP {
                 return mcpError(err);
             }
         });
+    }
+    /*  register the MCP tools reading an attachment of, deleting, and renaming a task plan  */
+    registerPlanTools(mcp) {
         /*  task attachment  */
         mcp.registerTool("ase_task_attachment", {
             title: "ASE task attachment",
@@ -1482,6 +1501,9 @@ export class TaskMCP {
                 return mcpError(err);
             }
         });
+    }
+    /*  register the MCP tools getting/setting the status of and requesting the processing of a task plan  */
+    registerStatusTools(mcp) {
         /*  task status get/set  */
         mcp.registerTool("ase_task_status", {
             title: "ASE task status get/set",
@@ -1539,6 +1561,9 @@ export class TaskMCP {
                 return mcpError(err);
             }
         });
+    }
+    /*  register the MCP tool assigning a task plan (with AI usage accounting)  */
+    registerAssignTool(mcp) {
         /*  task assignment (with AI usage accounting)  */
         mcp.registerTool("ase_task_assign", {
             title: "ASE task assignment",
@@ -1586,6 +1611,9 @@ export class TaskMCP {
                 return mcpError(err);
             }
         });
+    }
+    /*  register the MCP tools generating task ids and getting/setting the active task id  */
+    registerIdTools(mcp) {
         /*  task id generation  */
         mcp.registerTool("ase_task_newid", {
             title: "ASE task id generation",

package/dst/ase-util-compat.js

@@ -50,7 +50,7 @@ export default class CompatCommand {
         });
     }
 }
-/*  MCP tool "ase_compat" — lets the skill call this without a Bash tool  */
+/*  MCP tool "ase_compat" -- lets the skill call this without a Bash tool  */
 export class CompatMCP {
     register(mcp) {
         mcp.registerTool("ase_compat", {