chapterhouse 0.13.0 → 0.13.1

package/dist/copilot/agents.js

@@ -23,6 +23,7 @@ const agentFrontmatterSchema = z.object({
     model: z.string().min(1),
     skills: z.array(z.string()).optional(),
     tools: z.array(z.string()).optional(),
+    management_tools: z.array(z.string()).optional(),
     mcpServers: z.array(z.string()).optional(),
     allowed_paths: z.array(z.string()).optional(),
     persistent: z.union([z.boolean(), z.string()]).optional().transform((value) => {
@@ -83,6 +84,7 @@ export function parseAgentMdOrThrow(content, slug) {
         scope: fm.scope,
         skills: fm.skills,
         tools: fm.tools,
+        managementTools: fm.management_tools,
         mcpServers: fm.mcpServers,
         allowedPaths: fm.allowed_paths,
         systemMessage: body,
@@ -374,13 +376,20 @@ export function filterToolsForAgent(agent, allTools) {
     if (agent.tools && agent.tools.length > 0) {
         // Agent specifies an explicit allowlist — give those + wiki tools
         const allowed = new Set([...agent.tools, ...WIKI_TOOL_NAMES]);
-        return allTools.filter((t) => allowed.has(t.name) && !(agent.persistent && MANAGEMENT_TOOL_NAMES.has(t.name)));
+        return allTools.filter((t) => allowed.has(t.name));
     }
-    // Default: all tools except management (only @chapterhouse gets those)
+    // Default: all tools except management (only @chapterhouse gets those by default)
     if (agent.slug === "chapterhouse") {
         return allTools;
     }
-    return allTools.filter((t) => !MANAGEMENT_TOOL_NAMES.has(t.name));
+    const baseTools = allTools.filter((t) => !MANAGEMENT_TOOL_NAMES.has(t.name));
+    // Agents can opt into specific management tools via management_tools: in their config
+    if (agent.managementTools && agent.managementTools.length > 0) {
+        const optedIn = new Set(agent.managementTools);
+        const extra = allTools.filter((t) => MANAGEMENT_TOOL_NAMES.has(t.name) && optedIn.has(t.name));
+        return [...baseTools, ...extra];
+    }
+    return baseTools;
 }
 /** Filter MCP servers based on agent config. */
 export function filterMcpServersForAgent(agent, allMcpServers) {

package/dist/wiki/frontmatter.js

@@ -1,7 +1,11 @@
 import { normalizeWikiPath } from "./path-utils.js";
 import { DEFAULT_SCHEMA, load } from "js-yaml";
 const FRONTMATTER_RE = /^---\s*\n([\s\S]*?)\n---\s*\n?/;
-const SUMMARY_MARKDOWN_RE = /(\*\*|__|[_*`~]|^\s*#+\s|\[[^\]]+\]\([^)]+\)|!\[[^\]]*\]\([^)]+\)|^\s*>)/m;
+// Markdown formatting that must not appear in a one-line plain-text summary.
+// Detects *actual* formatting — paired or line-anchored — not lone characters:
+// a bare underscore/asterisk inside a word (e.g. `ask_user_question`) is plain
+// text, not emphasis, and must not be rejected.
+const SUMMARY_MARKDOWN_RE = /(\*\*[^*]+\*\*|__[^_]+__|\*[^\s*][^*]*\*|(?<![A-Za-z0-9])_[^\s_][^_]*_(?![A-Za-z0-9])|`[^`]+`|~~[^~]+~~|^\s*#+\s|\[[^\]]+\]\([^)]+\)|!\[[^\]]*\]\([^)]+\)|^\s*>)/m;
 const FRONTMATTER_TEMPLATE = `---\ntitle: <title>\nsummary: <plain-text one-line summary, max 200 chars>\nupdated: YYYY-MM-DD\ntags: []\nrelated: []\n---`;
 const PROJECT_RULES_HARD_DEFAULTS = {
     auto_pr: true,
@@ -159,14 +163,22 @@ export function validateWikiFrontmatter(content, options = {}) {
     }
     else {
         const summary = parsed.summary.trim();
-        if (summary.length > 200 ||
-            summary.includes("\n") ||
-            summary.includes("\r") ||
-            SUMMARY_MARKDOWN_RE.test(summary)) {
+        const markdownMatch = summary.match(SUMMARY_MARKDOWN_RE);
+        let reason;
+        if (summary.length > 200) {
+            reason = `invalid 'summary' — it is ${summary.length} characters; the maximum is 200`;
+        }
+        else if (summary.includes("\n") || summary.includes("\r")) {
+            reason = "invalid 'summary' — it must be a single line";
+        }
+        else if (markdownMatch) {
+            reason = `invalid 'summary' — it must be plain text, but contains markdown formatting (${JSON.stringify(markdownMatch[0])})`;
+        }
+        if (reason) {
             errors.push({
                 rule: "invalid-summary",
                 field: "summary",
-                message: formatFrontmatterMessage("invalid 'summary' (use plain text, one line, max 200 chars)"),
+                message: formatFrontmatterMessage(reason),
             });
         }
     }

package/dist/wiki/frontmatter.test.js

@@ -293,4 +293,44 @@ test("validateAndBackfillFrontmatter backfilled last_updated is valid ISO timest
     const ts = match[1].trim();
     assert.doesNotThrow(() => new Date(ts).toISOString(), `${ts} should be valid ISO timestamp`);
 });
+// ── Issue #458: summary markdown detection must not flag plain text ──────────
+test("validateWikiFrontmatter accepts a plain-text summary containing technical identifiers", async () => {
+    const { validateWikiFrontmatter } = await loadFrontmatterModule();
+    const result = validateWikiFrontmatter(`---
+title: Ask User Question
+summary: Design notes for the ask_user_question tool, stored under ~/.chapterhouse
+updated: 2026-05-19
+---
+
+# Ask User Question
+`);
+    assert.equal(result.valid, true, JSON.stringify(result.errors));
+});
+test("validateWikiFrontmatter still rejects genuine markdown formatting in summaries", async () => {
+    const { validateWikiFrontmatter } = await loadFrontmatterModule();
+    const markdownSummaries = [
+        "**bold text**",
+        "__bold text__",
+        "an *italic* word",
+        "an _italic_ word",
+        "a `code` span",
+        "a ~~struck~~ phrase",
+        "[a link](https://example.com)",
+    ];
+    for (const summary of markdownSummaries) {
+        const result = validateWikiFrontmatter(`---\ntitle: T\nsummary: ${summary}\nupdated: 2026-05-19\n---\n\n# T\n`);
+        assert.equal(result.valid, false, `expected "${summary}" to be rejected`);
+        assert.equal(result.errors[0]?.rule, "invalid-summary", `"${summary}" should fail invalid-summary`);
+    }
+});
+test("validateWikiFrontmatter invalid-summary message identifies the specific problem", async () => {
+    const { validateWikiFrontmatter } = await loadFrontmatterModule();
+    const markdown = validateWikiFrontmatter(`---\ntitle: T\nsummary: a \`code\` span\nupdated: 2026-05-19\n---\n\n# T\n`);
+    assert.equal(markdown.valid, false);
+    assert.match(markdown.errors[0].message, /markdown formatting/i);
+    assert.match(markdown.errors[0].message, /`code`/);
+    const tooLong = validateWikiFrontmatter(`---\ntitle: T\nsummary: ${"x".repeat(201)}\nupdated: 2026-05-19\n---\n\n# T\n`);
+    assert.equal(tooLong.valid, false);
+    assert.match(tooLong.errors[0].message, /200/);
+});
 //# sourceMappingURL=frontmatter.test.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chapterhouse",
-  "version": "0.13.0",
+  "version": "0.13.1",
   "description": "Chapterhouse — a team-level AI assistant for engineering teams, built on the GitHub Copilot SDK. Web UI only.",
   "bin": {
     "chapterhouse": "dist/cli.js"