@agentteams/cli 0.1.56 → 0.1.58-dev.224

package/dist/commands/convention.js

@@ -1,26 +1,26 @@
-import { existsSync, mkdirSync, readFileSync, rmSync, unlinkSync } from "node:fs";
-import { atomicWriteFileSync } from "../utils/atomicWrite.js";
-import { basename, join, relative, resolve, sep } from "node:path";
-import httpClient from "../utils/httpClient.js";
-import { isAxiosError } from "axios";
-import matter from "gray-matter";
-import { diffLines, createTwoFilesPatch } from "diff";
-import { loadConfig, findProjectConfig } from "../utils/config.js";
-import { withSpinner } from "../utils/spinner.js";
-import { withoutJsonContentType } from "../utils/httpHeaders.js";
-const CONVENTION_DIR = ".agentteams";
-const LEGACY_CONVENTION_DOWNLOAD_DIR = "conventions";
-const CONVENTION_INDEX_FILE = "convention.md";
-const CONVENTION_MANIFEST_FILE = "conventions.manifest.json";
+import { existsSync, mkdirSync, readFileSync, rmSync, unlinkSync } from 'node:fs';
+import { atomicWriteFileSync } from '../utils/atomicWrite.js';
+import { basename, join, relative, resolve, sep } from 'node:path';
+import httpClient from '../utils/httpClient.js';
+import { isAxiosError } from 'axios';
+import matter from 'gray-matter';
+import { diffLines, createTwoFilesPatch } from 'diff';
+import { loadConfig, findProjectConfig } from '../utils/config.js';
+import { withSpinner } from '../utils/spinner.js';
+import { withoutJsonContentType } from '../utils/httpHeaders.js';
+const CONVENTION_DIR = '.agentteams';
+const LEGACY_CONVENTION_DOWNLOAD_DIR = 'conventions';
+const CONVENTION_INDEX_FILE = 'convention.md';
+const CONVENTION_MANIFEST_FILE = 'conventions.manifest.json';
 function findProjectRoot(cwd) {
     const configPath = findProjectConfig(cwd ?? process.cwd());
     if (!configPath)
         return null;
     // configPath = /path/.agentteams/config.json → resolve up 2 levels to project root
-    return resolve(configPath, "..", "..");
+    return resolve(configPath, '..', '..');
 }
 function getApiBaseUrl(apiUrl) {
-    return apiUrl.endsWith("/") ? apiUrl.slice(0, -1) : apiUrl;
+    return apiUrl.endsWith('/') ? apiUrl.slice(0, -1) : apiUrl;
 }
 function getApiConfigOrThrow(options) {
     const config = options?.config ?? loadConfig();
@@ -31,13 +31,13 @@ function getApiConfigOrThrow(options) {
         config,
         apiUrl: getApiBaseUrl(config.apiUrl),
         headers: {
-            "X-API-Key": config.apiKey,
-            "Content-Type": "application/json",
+            'X-API-Key': config.apiKey,
+            'Content-Type': 'application/json',
         },
     };
 }
 function normalizeRelativePath(input) {
-    return input.replaceAll("\\", "/");
+    return input.replaceAll('\\', '/');
 }
 function resolveConventionFileAbsolutePath(projectRoot, cwd, fileInput) {
     // If absolute path, keep as-is.
@@ -59,7 +59,7 @@ function resolveConventionFileAbsolutePath(projectRoot, cwd, fileInput) {
 function validatePathBoundary(absolutePath, projectRoot) {
     const normalized = resolve(absolutePath);
     if (!normalized.startsWith(resolve(projectRoot) + sep)) {
-        throw new Error("Path traversal detected: file must be within project root");
+        throw new Error('Path traversal detected: file must be within project root');
     }
     return normalized;
 }
@@ -71,7 +71,7 @@ function loadManifestOrThrow(projectRoot) {
     if (!existsSync(manifestPath)) {
         throw new Error(`Download manifest not found: ${manifestPath}\nRun 'agentteams convention download' first.`);
     }
-    const raw = readFileSync(manifestPath, "utf-8");
+    const raw = readFileSync(manifestPath, 'utf-8');
     const parsed = JSON.parse(raw);
     if (parsed?.version !== 1 || !Array.isArray(parsed.entries)) {
         throw new Error(`Invalid manifest format: ${manifestPath}`);
@@ -87,7 +87,7 @@ function loadManifestOrCreate(projectRoot) {
             entries: [],
         };
     }
-    const raw = readFileSync(manifestPath, "utf-8");
+    const raw = readFileSync(manifestPath, 'utf-8');
     const parsed = JSON.parse(raw);
     if (parsed?.version !== 1 || !Array.isArray(parsed.entries)) {
         throw new Error(`Invalid manifest format: ${manifestPath}`);
@@ -96,7 +96,7 @@ function loadManifestOrCreate(projectRoot) {
 }
 function writeManifest(projectRoot, manifest) {
     const manifestPath = buildManifestPath(projectRoot);
-    atomicWriteFileSync(manifestPath, JSON.stringify(manifest, null, 2) + "\n", "utf-8");
+    atomicWriteFileSync(manifestPath, JSON.stringify(manifest, null, 2) + '\n', 'utf-8');
 }
 function toFileList(input) {
     return Array.isArray(input) ? input : [input];
@@ -106,21 +106,19 @@ function hasAnyDiff(a, b) {
     return parts.some((p) => p.added || p.removed);
 }
 function createUnifiedDiff(fileLabel, serverText, localText) {
-    return createTwoFilesPatch(`${fileLabel} (server)`, `${fileLabel} (local)`, serverText, localText, "", "", { context: 3 });
+    return createTwoFilesPatch(`${fileLabel} (server)`, `${fileLabel} (local)`, serverText, localText, '', '', {
+        context: 3,
+    });
 }
 function toOptionalString(value) {
-    return typeof value === "string" ? value : undefined;
+    return typeof value === 'string' ? value : undefined;
 }
 function fileNameToTitle(fileName) {
-    return fileName
-        .replace(/\.md$/i, "")
-        .replace(/[-_]+/g, " ")
-        .replace(/\s+/g, " ")
-        .trim();
+    return fileName.replace(/\.md$/i, '').replace(/[-_]+/g, ' ').replace(/\s+/g, ' ').trim();
 }
 function parseCategoryFromAgentteamsPath(fileRelativePath) {
     const normalized = normalizeRelativePath(fileRelativePath);
-    const parts = normalized.split("/");
+    const parts = normalized.split('/');
     const agentteamsIndex = parts.indexOf(CONVENTION_DIR);
     if (agentteamsIndex === -1) {
         throw new Error(`Convention create requires a file under ${CONVENTION_DIR}/<category>/: ${fileRelativePath}`);
@@ -129,7 +127,7 @@ function parseCategoryFromAgentteamsPath(fileRelativePath) {
     if (!category || category.length === 0) {
         throw new Error(`Convention create requires a category directory under ${CONVENTION_DIR}/: ${fileRelativePath}`);
     }
-    if (category === "platform" || category === "active-plan") {
+    if (category === 'platform' || category === 'active-plan') {
         throw new Error(`Convention create does not allow reserved directories under ${CONVENTION_DIR}/: ${category}`);
     }
     return category;
@@ -140,14 +138,17 @@ async function fetchAllConventions(apiUrl, projectId, headers) {
     let totalPages;
     const items = [];
     while (true) {
-        const response = await httpClient.get(`${apiUrl}/api/projects/${projectId}/conventions`, { headers, params: { page, pageSize } });
+        const response = await httpClient.get(`${apiUrl}/api/projects/${projectId}/conventions`, {
+            headers,
+            params: { page, pageSize },
+        });
         const data = response.data?.data;
         if (!Array.isArray(data)) {
             break;
         }
         items.push(...data);
         const meta = response.data?.meta;
-        if (typeof meta?.totalPages === "number") {
+        if (typeof meta?.totalPages === 'number') {
             totalPages = meta.totalPages;
         }
         if (totalPages !== undefined) {
@@ -167,32 +168,32 @@ async function fetchConventionsWithContent(apiUrl, projectId, headers) {
     const response = await httpClient.get(`${apiUrl}/api/projects/${projectId}/conventions/download-all`, { headers });
     const data = response.data?.data;
     if (!Array.isArray(data)) {
-        throw new Error("Invalid download-all response format");
+        throw new Error('Invalid download-all response format');
     }
     return data;
 }
 async function fetchPlatformGuidesHash(apiUrl, headers) {
     const response = await httpClient.get(`${apiUrl}/api/platform/guides/hash`, { headers });
     const hash = response.data?.data?.hash;
-    if (typeof hash !== "string" || hash.length === 0) {
-        throw new Error("Invalid platform guides hash response format");
+    if (typeof hash !== 'string' || hash.length === 0) {
+        throw new Error('Invalid platform guides hash response format');
     }
     return hash;
 }
 function toConventionName(convention) {
-    const title = typeof convention.title === "string" ? convention.title.trim() : "";
+    const title = typeof convention.title === 'string' ? convention.title.trim() : '';
     if (title.length > 0)
         return title;
-    const fileName = typeof convention.fileName === "string" ? convention.fileName.trim() : "";
+    const fileName = typeof convention.fileName === 'string' ? convention.fileName.trim() : '';
     if (fileName.length > 0)
         return fileName;
     return convention.id;
 }
 function toConventionNameFromManifest(entry) {
-    const title = typeof entry.title === "string" ? entry.title.trim() : "";
+    const title = typeof entry.title === 'string' ? entry.title.trim() : '';
     if (title.length > 0)
         return title;
-    const fileName = typeof entry.fileName === "string" ? entry.fileName.trim() : "";
+    const fileName = typeof entry.fileName === 'string' ? entry.fileName.trim() : '';
     if (fileName.length > 0)
         return fileName;
     return entry.conventionId;
@@ -205,7 +206,7 @@ function toOptionalStringOrNullIfPresent(data, key) {
     if (value === null) {
         return null;
     }
-    if (typeof value === "string") {
+    if (typeof value === 'string') {
         const trimmed = value.trim();
         return trimmed.length > 0 ? trimmed : null;
     }
@@ -215,15 +216,15 @@ export async function conventionShow() {
     const { config, apiUrl, headers } = getApiConfigOrThrow();
     const conventions = await fetchConventionsWithContent(apiUrl, config.projectId, headers);
     if (!conventions || conventions.length === 0) {
-        throw new Error("No conventions found for this project. Create one via the web dashboard first.");
+        throw new Error('No conventions found for this project. Create one via the web dashboard first.');
     }
     const sections = [];
     for (const convention of conventions) {
-        const contentMarkdown = typeof convention.contentMarkdown === "string" ? convention.contentMarkdown : "";
-        const sectionHeader = `# ${convention.title ?? "untitled"}\ncategory: ${convention.category ?? "uncategorized"}\nid: ${convention.id}`;
+        const contentMarkdown = typeof convention.contentMarkdown === 'string' ? convention.contentMarkdown : '';
+        const sectionHeader = `# ${convention.title ?? 'untitled'}\ncategory: ${convention.category ?? 'uncategorized'}\nid: ${convention.id}`;
         sections.push(`${sectionHeader}\n\n${contentMarkdown}`);
     }
-    return sections.join("\n\n---\n\n");
+    return sections.join('\n\n---\n\n');
 }
 export async function checkConventionFreshness(apiUrl, projectId, headers, projectRoot) {
     const manifestPath = buildManifestPath(projectRoot);
@@ -235,9 +236,9 @@ export async function checkConventionFreshness(apiUrl, projectId, headers, proje
     }
     const manifest = loadManifestOrThrow(projectRoot);
     const currentPlatformGuidesHash = await fetchPlatformGuidesHash(apiUrl, headers);
-    const platformGuidesChanged = typeof manifest.platformGuidesHash === "string"
-        && manifest.platformGuidesHash.length > 0
-        && manifest.platformGuidesHash !== currentPlatformGuidesHash;
+    const platformGuidesChanged = typeof manifest.platformGuidesHash === 'string' &&
+        manifest.platformGuidesHash.length > 0 &&
+        manifest.platformGuidesHash !== currentPlatformGuidesHash;
     const serverConventions = await fetchAllConventions(apiUrl, projectId, headers);
     const serverById = new Map(serverConventions.map((item) => [item.id, item]));
     const localById = new Map(manifest.entries.map((entry) => [entry.conventionId, entry]));
@@ -247,18 +248,18 @@ export async function checkConventionFreshness(apiUrl, projectId, headers, proje
         if (!local) {
             conventionChanges.push({
                 id: serverConvention.id,
-                type: "new",
+                type: 'new',
                 title: toConventionName(serverConvention),
                 fileName: serverConvention.fileName ?? undefined,
             });
             continue;
         }
-        if (typeof serverConvention.updatedAt === "string"
-            && typeof local.updatedAt === "string"
-            && serverConvention.updatedAt !== local.updatedAt) {
+        if (typeof serverConvention.updatedAt === 'string' &&
+            typeof local.updatedAt === 'string' &&
+            serverConvention.updatedAt !== local.updatedAt) {
             conventionChanges.push({
                 id: serverConvention.id,
-                type: "updated",
+                type: 'updated',
                 title: toConventionName(serverConvention),
                 fileName: serverConvention.fileName ?? local.fileName,
             });
@@ -269,7 +270,7 @@ export async function checkConventionFreshness(apiUrl, projectId, headers, proje
             continue;
         conventionChanges.push({
             id: localEntry.conventionId,
-            type: "deleted",
+            type: 'deleted',
             title: toConventionNameFromManifest(localEntry),
             fileName: localEntry.fileName,
         });
@@ -279,6 +280,56 @@ export async function checkConventionFreshness(apiUrl, projectId, headers, proje
         conventionChanges,
     };
 }
+export function buildStatusSummary(result) {
+    const parts = [];
+    if (result.platformGuidesChanged)
+        parts.push('platform guides');
+    const counts = { new: 0, updated: 0, deleted: 0 };
+    for (const change of result.conventionChanges)
+        counts[change.type] += 1;
+    if (counts.new > 0)
+        parts.push(`${counts.new} new`);
+    if (counts.updated > 0)
+        parts.push(`${counts.updated} updated`);
+    if (counts.deleted > 0)
+        parts.push(`${counts.deleted} deleted`);
+    if (parts.length === 0)
+        return '✓ Conventions up to date';
+    return `⚠ Updates available (${parts.join(', ')}). Run 'agentteams convention download' to sync.`;
+}
+/**
+ * Read-only freshness check exposed as `agentteams convention status`.
+ *
+ * Compares the local download manifest against the server and reports whether an
+ * update is available — it never downloads or mutates anything. Degrades gracefully
+ * (exit 0, updateAvailable=false) when the project is not configured or has no local
+ * conventions yet, so callers can safely "check then skip when unavailable".
+ */
+export async function conventionStatus(options) {
+    const config = options?.config ?? loadConfig();
+    const projectRoot = findProjectRoot(options?.cwd);
+    // Not configured or no local conventions → nothing to compare; treat as up to date.
+    if (!config || !projectRoot) {
+        return {
+            updateAvailable: false,
+            platformGuidesChanged: false,
+            conventionChanges: [],
+            summary: '✓ Conventions up to date',
+        };
+    }
+    const apiUrl = getApiBaseUrl(config.apiUrl);
+    const headers = {
+        'X-API-Key': config.apiKey,
+        'Content-Type': 'application/json',
+    };
+    const freshness = await checkConventionFreshness(apiUrl, config.projectId, headers, projectRoot);
+    return {
+        updateAvailable: freshness.platformGuidesChanged || freshness.conventionChanges.length > 0,
+        platformGuidesChanged: freshness.platformGuidesChanged,
+        conventionChanges: freshness.conventionChanges,
+        summary: buildStatusSummary(freshness),
+    };
+}
 export async function conventionList() {
     const { config, apiUrl, headers } = getApiConfigOrThrow();
     const conventions = await fetchAllConventions(apiUrl, config.projectId, headers);
@@ -299,33 +350,31 @@ export async function conventionList() {
             page: 1,
             pageSize: conventions.length,
             totalPages: 1,
-        }
+        },
     };
 }
 function toSafeFileName(input) {
     return input
         .toLowerCase()
-        .replace(/[^a-z0-9]+/g, "-")
-        .replace(/^-+|-+$/g, "")
+        .replace(/[^a-z0-9]+/g, '-')
+        .replace(/^-+|-+$/g, '')
         .slice(0, 60);
 }
 function toSafeDirectoryName(input) {
     const normalized = toSafeFileName(input);
-    return normalized.length > 0 ? normalized : "uncategorized";
+    return normalized.length > 0 ? normalized : 'uncategorized';
 }
 function buildConventionFileName(convention) {
     if (convention.fileName && convention.fileName.trim().length > 0) {
         return convention.fileName.trim();
     }
-    const titleSegment = convention.title ? toSafeFileName(convention.title) : "";
-    const prefix = titleSegment.length > 0 ? titleSegment : "convention";
+    const titleSegment = convention.title ? toSafeFileName(convention.title) : '';
+    const prefix = titleSegment.length > 0 ? titleSegment : 'convention';
     return `${prefix}.md`;
 }
 function normalizeMarkdownFileName(input) {
     const trimmed = input.trim();
-    const base = trimmed.toLowerCase().endsWith('.md')
-        ? trimmed.slice(0, -3)
-        : trimmed;
+    const base = trimmed.toLowerCase().endsWith('.md') ? trimmed.slice(0, -3) : trimmed;
     const safeBase = toSafeFileName(base);
     const resolvedBase = safeBase.length > 0 ? safeBase : 'guide';
     return `${resolvedBase}.md`;
@@ -358,9 +407,7 @@ async function downloadPlatformGuides(projectRoot, apiUrl, headers) {
             const baseFileName = buildPlatformGuideFileName(guide);
             const seenCount = fileNameCount.get(baseFileName) ?? 0;
             fileNameCount.set(baseFileName, seenCount + 1);
-            const fileName = seenCount === 0
-                ? baseFileName
-                : baseFileName.replace(/\.md$/, `-${seenCount + 1}.md`);
+            const fileName = seenCount === 0 ? baseFileName : baseFileName.replace(/\.md$/, `-${seenCount + 1}.md`);
             const filePath = join(baseDir, fileName);
             atomicWriteFileSync(filePath, guide.content, 'utf-8');
             written += 1;
@@ -375,22 +422,24 @@ async function downloadPlatformGuides(projectRoot, apiUrl, headers) {
     }
 }
 async function downloadReportingTemplate(projectRoot, config, apiUrl, headers) {
-    const agentConfigResponse = await httpClient.get(`${apiUrl}/api/projects/${config.projectId}/agent-configs`, { headers });
+    const agentConfigResponse = await httpClient.get(`${apiUrl}/api/projects/${config.projectId}/agent-configs`, {
+        headers,
+    });
     const agentConfigs = agentConfigResponse.data?.data;
     if (!Array.isArray(agentConfigs) || agentConfigs.length === 0) {
         return false;
     }
     const firstAgentConfig = agentConfigs[0];
-    if (!firstAgentConfig?.id || typeof firstAgentConfig.id !== "string") {
+    if (!firstAgentConfig?.id || typeof firstAgentConfig.id !== 'string') {
         return false;
     }
     const templateResponse = await httpClient.get(`${apiUrl}/api/projects/${config.projectId}/agent-configs/${firstAgentConfig.id}/convention`, { headers });
     const content = templateResponse.data?.data?.content;
-    if (typeof content !== "string") {
+    if (typeof content !== 'string') {
         return false;
     }
     const conventionPath = join(projectRoot, CONVENTION_DIR, CONVENTION_INDEX_FILE);
-    atomicWriteFileSync(conventionPath, content, "utf-8");
+    atomicWriteFileSync(conventionPath, content, 'utf-8');
     return true;
 }
 export async function conventionDownload(options) {
@@ -414,7 +463,7 @@ export async function conventionDownload(options) {
         rmSync(legacyDir, { recursive: true, force: true });
         const categoryDirs = new Set();
         for (const convention of conventionList) {
-            const categoryName = typeof convention.category === "string" ? convention.category : "";
+            const categoryName = typeof convention.category === 'string' ? convention.category : '';
             categoryDirs.add(toSafeDirectoryName(categoryName));
         }
         for (const categoryDir of categoryDirs) {
@@ -430,18 +479,16 @@ export async function conventionDownload(options) {
             entries: [],
         };
         for (const convention of conventionList) {
-            const contentMarkdown = typeof convention.contentMarkdown === "string" ? convention.contentMarkdown : "";
+            const contentMarkdown = typeof convention.contentMarkdown === 'string' ? convention.contentMarkdown : '';
             const baseFileName = buildConventionFileName(convention);
-            const categoryName = typeof convention.category === "string" ? convention.category : "";
+            const categoryName = typeof convention.category === 'string' ? convention.category : '';
             const categoryDir = toSafeDirectoryName(categoryName);
             const duplicateKey = `${categoryDir}/${baseFileName}`;
             const seenCount = fileNameCount.get(duplicateKey) ?? 0;
             fileNameCount.set(duplicateKey, seenCount + 1);
-            const fileName = seenCount === 0
-                ? baseFileName
-                : baseFileName.replace(/\.md$/, `-${seenCount + 1}.md`);
+            const fileName = seenCount === 0 ? baseFileName : baseFileName.replace(/\.md$/, `-${seenCount + 1}.md`);
             const filePath = join(projectRoot, CONVENTION_DIR, categoryDir, fileName);
-            atomicWriteFileSync(filePath, contentMarkdown, "utf-8");
+            atomicWriteFileSync(filePath, contentMarkdown, 'utf-8');
             manifest.entries.push({
                 conventionId: String(convention.id),
                 fileRelativePath: normalizeRelativePath(relative(projectRoot, filePath)),
@@ -464,14 +511,12 @@ export async function conventionDownload(options) {
                 : '';
             return `Convention sync completed.\nUpdated ${CONVENTION_DIR}/${CONVENTION_INDEX_FILE}\nNo project conventions found.${platformLine}`;
         }
-        throw new Error("No conventions found for this project. Create one via the web dashboard first.");
+        throw new Error('No conventions found for this project. Create one via the web dashboard first.');
     }
-    const reportingLine = hasReportingTemplate
-        ? `Updated ${CONVENTION_DIR}/${CONVENTION_INDEX_FILE}\n`
-        : "";
+    const reportingLine = hasReportingTemplate ? `Updated ${CONVENTION_DIR}/${CONVENTION_INDEX_FILE}\n` : '';
     const platformLine = platformGuideCount > 0
         ? `Downloaded ${platformGuideCount} platform guide file(s) into ${CONVENTION_DIR}/platform\n`
-        : "";
+        : '';
     return `Convention sync completed.\n${reportingLine}${platformLine}Downloaded ${conventions.length} file(s) into category directories under ${CONVENTION_DIR}`;
 }
 function parseConventionMarkdown(fileRelativePath, markdown) {
@@ -501,7 +546,7 @@ export async function conventionCreate(options) {
         const fileRelativePath = normalizeRelativePath(relative(projectRoot, absolutePath));
         const category = parseCategoryFromAgentteamsPath(fileRelativePath);
         const fileName = basename(absolutePath);
-        if (!fileName.toLowerCase().endsWith(".md")) {
+        if (!fileName.toLowerCase().endsWith('.md')) {
             throw new Error(`Convention create requires a .md file: ${fileRelativePath}`);
         }
         const existingEntry = manifest.entries.find((e) => e.fileRelativePath === fileRelativePath);
@@ -509,10 +554,10 @@ export async function conventionCreate(options) {
             throw new Error(`File is already tracked in the manifest (use update instead): ${fileRelativePath}\n` +
                 `- conventionId: ${existingEntry.conventionId}`);
         }
-        const localMarkdown = readFileSync(absolutePath, "utf-8");
+        const localMarkdown = readFileSync(absolutePath, 'utf-8');
         const parsed = parseConventionMarkdown(fileRelativePath, localMarkdown);
         const frontmatter = (parsed.data ?? {});
-        const bodyMarkdown = String(parsed.content ?? "");
+        const bodyMarkdown = String(parsed.content ?? '');
         const content = bodyMarkdown;
         const title = toOptionalString(frontmatter.title)?.trim() || fileNameToTitle(fileName);
         const payload = {
@@ -528,14 +573,14 @@ export async function conventionCreate(options) {
             payload.trigger = trigger;
         if (description)
             payload.description = description;
-        if (typeof agentInstruction === "string" && agentInstruction.trim().length > 0) {
+        if (typeof agentInstruction === 'string' && agentInstruction.trim().length > 0) {
             payload.agentInstruction = agentInstruction.trimEnd();
         }
         const response = await withSpinner(`Creating convention for ${fileRelativePath}...`, () => httpClient.post(`${apiUrl}/api/projects/${config.projectId}/conventions`, payload, { headers }));
         const created = response.data?.data;
-        const createdId = typeof created?.id === "string" ? created.id : "unknown";
-        const createdUpdatedAt = typeof created?.updatedAt === "string" ? created.updatedAt : undefined;
-        const createdWebUrl = typeof created?.webUrl === "string" ? created.webUrl : undefined;
+        const createdId = typeof created?.id === 'string' ? created.id : 'unknown';
+        const createdUpdatedAt = typeof created?.updatedAt === 'string' ? created.updatedAt : undefined;
+        const createdWebUrl = typeof created?.webUrl === 'string' ? created.webUrl : undefined;
         const now = new Date().toISOString();
         manifest.generatedAt = now;
         manifest.entries.push({
@@ -558,7 +603,7 @@ export async function conventionCreate(options) {
         results.push(`[OK] ${CONVENTION_DIR}/${CONVENTION_MANIFEST_FILE}: Updated.`);
         results.push(`[NEXT] Run 'agentteams convention download' to refresh convention.md and canonical markdown.`);
     }
-    return results.join("\n");
+    return results.join('\n');
 }
 export async function conventionUpdate(options) {
     const { config, apiUrl, headers } = getApiConfigOrThrow(options);
@@ -583,13 +628,13 @@ export async function conventionUpdate(options) {
             throw new Error(`Only downloaded convention files can be updated: ${fileInput}\n` +
                 `- resolved: ${fileRelativePath}\n` +
                 `Run 'agentteams convention download' first, or pass a file path listed in the manifest.\n` +
-                (available.length > 0 ? `Examples (partial):\n- ${available.join("\n- ")}` : ""));
+                (available.length > 0 ? `Examples (partial):\n- ${available.join('\n- ')}` : ''));
         }
         const conventionId = manifestEntry.conventionId;
         const [serverDetail, serverMarkdown, localMarkdown] = await withSpinner(`Preparing update for ${fileRelativePath}...`, async () => {
             const detailResponse = await httpClient.get(`${apiUrl}/api/projects/${config.projectId}/conventions/${conventionId}`, { headers });
-            const downloadResponse = await httpClient.get(`${apiUrl}/api/projects/${config.projectId}/conventions/${conventionId}/download`, { headers, responseType: "text" });
-            const local = readFileSync(absolutePath, "utf-8");
+            const downloadResponse = await httpClient.get(`${apiUrl}/api/projects/${config.projectId}/conventions/${conventionId}/download`, { headers, responseType: 'text' });
+            const local = readFileSync(absolutePath, 'utf-8');
             return [detailResponse.data?.data, String(downloadResponse.data), local];
         });
         if (!hasAnyDiff(serverMarkdown, localMarkdown)) {
@@ -604,18 +649,18 @@ export async function conventionUpdate(options) {
         }
         const parsed = parseConventionMarkdown(fileRelativePath, localMarkdown);
         const frontmatter = (parsed.data ?? {});
-        const bodyMarkdown = String(parsed.content ?? "");
+        const bodyMarkdown = String(parsed.content ?? '');
         const content = bodyMarkdown;
-        if (typeof serverDetail?.updatedAt !== "string" || serverDetail.updatedAt.length === 0) {
+        if (typeof serverDetail?.updatedAt !== 'string' || serverDetail.updatedAt.length === 0) {
             throw new Error(`[ERROR] ${fileRelativePath}: Server response is missing updatedAt.`);
         }
         const payload = {
             updatedAt: serverDetail.updatedAt,
             content,
         };
-        const trigger = toOptionalStringOrNullIfPresent(frontmatter, "trigger");
-        const description = toOptionalStringOrNullIfPresent(frontmatter, "description");
-        const agentInstruction = toOptionalStringOrNullIfPresent(frontmatter, "agentInstruction");
+        const trigger = toOptionalStringOrNullIfPresent(frontmatter, 'trigger');
+        const description = toOptionalStringOrNullIfPresent(frontmatter, 'description');
+        const agentInstruction = toOptionalStringOrNullIfPresent(frontmatter, 'agentInstruction');
         if (trigger !== undefined)
             payload.trigger = trigger;
         if (description !== undefined)
@@ -624,12 +669,10 @@ export async function conventionUpdate(options) {
             payload.agentInstruction = agentInstruction;
         const updatedResponse = await withSpinner(`Uploading ${fileRelativePath}...`, () => httpClient.put(`${apiUrl}/api/projects/${config.projectId}/conventions/${conventionId}`, payload, { headers }));
         const newUpdatedAt = updatedResponse.data?.data?.updatedAt;
-        const newWebUrl = typeof updatedResponse.data?.data?.webUrl === "string"
-            ? updatedResponse.data.data.webUrl
-            : undefined;
+        const newWebUrl = typeof updatedResponse.data?.data?.webUrl === 'string' ? updatedResponse.data.data.webUrl : undefined;
         const now = new Date().toISOString();
         manifestEntry.lastUploadedAt = now;
-        if (typeof newUpdatedAt === "string") {
+        if (typeof newUpdatedAt === 'string') {
             manifestEntry.lastKnownUpdatedAt = newUpdatedAt;
         }
         writeManifest(projectRoot, manifest);
@@ -638,7 +681,7 @@ export async function conventionUpdate(options) {
             results.push(`webUrl: ${newWebUrl}`);
         }
     }
-    return results.join("\n\n");
+    return results.join('\n\n');
 }
 export async function conventionDelete(options) {
     const { config, apiUrl, headers } = getApiConfigOrThrow(options);
@@ -663,7 +706,7 @@ export async function conventionDelete(options) {
             throw new Error(`Only downloaded convention files can be deleted: ${fileInput}\n` +
                 `- resolved: ${fileRelativePath}\n` +
                 `Run 'agentteams convention download' first, or pass a file path listed in the manifest.\n` +
-                (available.length > 0 ? `Examples (partial):\n- ${available.join("\n- ")}` : ""));
+                (available.length > 0 ? `Examples (partial):\n- ${available.join('\n- ')}` : ''));
         }
         const entry = manifest.entries[entryIndex];
         const conventionId = entry.conventionId;
@@ -672,7 +715,9 @@ export async function conventionDelete(options) {
             results.push(`[DRY-RUN] ${fileRelativePath}: Planned only (no server delete).`);
             continue;
         }
-        await withSpinner(`Deleting convention for ${fileRelativePath}...`, () => httpClient.delete(`${apiUrl}/api/projects/${config.projectId}/conventions/${conventionId}`, { headers: withoutJsonContentType(headers) }));
+        await withSpinner(`Deleting convention for ${fileRelativePath}...`, () => httpClient.delete(`${apiUrl}/api/projects/${config.projectId}/conventions/${conventionId}`, {
+            headers: withoutJsonContentType(headers),
+        }));
         // After a successful server delete, also clean up local files/manifest.
         try {
             unlinkSync(absolutePath);
@@ -684,6 +729,6 @@ export async function conventionDelete(options) {
         writeManifest(projectRoot, manifest);
         results.push(`[OK] ${fileRelativePath}: Deleted. (conventionId=${conventionId})`);
     }
-    return results.join("\n");
+    return results.join('\n');
 }
 //# sourceMappingURL=convention.js.map