codiedev 0.4.0 → 0.5.1

package/dist/cli.js

@@ -41,8 +41,12 @@ Artifacts:
                                          Fetch an artifact (stdout by default)
   codiedev promote <artifact-id>         Promote an auto-extracted artifact
                                          to an authored one
-  codiedev reverse-ticket <pr-url>       Generate a Jira ticket from a merged PR
-                                         (auto-matches spec + thread + transcript)
+  codiedev reverse-ticket [<pr-url>]     Generate a Jira ticket draft.
+                                         No URL = branch mode (uses your current
+                                         local git branch, no PR needed).
+                                         Draft lands in /portal/agentic-ticketing.
+  codiedev reverse-ticket --base <ref>   Override the base ref in branch mode
+                                         (defaults to origin/main).
 
 Messaging:
   codiedev ping <user> "<msg>" [--with <key>]
@@ -58,15 +62,26 @@ Other:
   codiedev docs                          Print the full usage guide
   codiedev version                       Show version
 
-Filename conventions:
-  spec-*.md      → spec          review-*.md    → review
-  decision-*.md  → decision      proposal-*.md  → proposal
-  bugfix-*.md    → bugfix        anything else  → note
+The 90% workflow:
+  1. Name your file 'spec-<topic>.md' and push.
+  2. Ping a teammate to pull review.
+  3. They edit, push back. Versions stack. Done.
+
+Other artifact types (rare — use the matching prefix):
+  review-*.md → review     decision-*.md → decision
+  proposal-*.md → proposal  bugfix-*.md → bugfix
+  anything else → note
+
+Skills (slash commands):
+  Add 'name:' + 'description:' YAML frontmatter to any pushed file
+  and it shows up as /<name> in Claude Code / Codex. Filename
+  prefix doesn't matter — frontmatter is the signal.
 
 Examples:
-  codiedev push docs/specs/spec-cart-clear.md
+  codiedev push spec-cart-clear.md          # the typical case
+  codiedev push portal-design.md            # with name:/description: → skill
   codiedev pull spec-cart-clear.md
-  codiedev ping maya "thoughts on batch vs parallel?" --with spec-cart-clear.md
+  codiedev ping maya "thoughts?" --with spec-cart-clear.md
   codiedev inbox --unread
   codiedev note "idempotency key is worth a follow-up"
 
@@ -92,17 +107,23 @@ codiedev connect — link your agent CLI to CodieDev
     push: `
 codiedev push — author or update an artifact
 
-  Usage:
-    codiedev push <file.md> [--type spec|review|decision|proposal|bugfix|note]
+  Default path:
+    codiedev push spec-<topic>.md
 
-  The file's basename becomes its key (unique per company). Pushing the
-  same key again creates a new version — full history is preserved.
+  The file's basename becomes its key (unique per company). Pushing
+  the same key again versions it — full history is preserved.
 
-  Type is inferred from the filename prefix unless you override with --type.
+  Other types: rename with the matching prefix (review-*, decision-*,
+  proposal-*, bugfix-*) or pass --type. Anything else lands as 'note'.
+
+  Skills (slash commands): add 'name:' + 'description:' YAML
+  frontmatter to the top of the file. It shows up as /<name> in
+  the Skills tab. Filename prefix doesn't matter for the skill role.
 
   Examples:
-    codiedev push spec-cart-clear.md
-    codiedev push docs/review-auth.md --type review
+    codiedev push spec-cart-clear.md            # the typical case
+    codiedev push portal-design.md              # with name:/description: → skill
+    codiedev push docs/review-auth.md
     codiedev push notes/random.md --type note
 `.trim(),
     pull: `
@@ -240,6 +261,25 @@ Artifacts have a status that auto-advances as PRs reference them:
 Artifacts stay editable at every stage, including after merge —
 lessons-learned and follow-up notes append as new versions.
 
+## Skills
+
+Default path:
+
+    1. Pick a name: e.g. portal-design
+    2. Top the file with frontmatter:
+         ---
+         name: portal-design
+         description: Use when reviewing or building portal UI.
+         ---
+         # body...
+    3. codiedev push portal-design.md
+
+It's now /portal-design in Claude Code / Codex.
+
+Filename prefix is irrelevant for skills — frontmatter is the only
+signal. The stored artifact type still follows the filename rule
+(so a skill in a spec file is both a 'spec' and a callable skill).
+
 ## Portal
 
 Everything you do in the CLI is visible at:

package/dist/commands/reverseTicket.js

@@ -33,6 +33,7 @@ function ghRaw(path) {
 }
 function parseArgs(args) {
     let prUrl;
+    let base;
     let forcedKey;
     let noTruncate = false;
     for (let i = 0; i < args.length; i++) {
@@ -43,6 +44,12 @@ function parseArgs(args) {
         else if (a.startsWith("--with=")) {
             forcedKey = a.slice("--with=".length);
         }
+        else if ((a === "--base" || a === "-b") && i + 1 < args.length) {
+            base = args[++i];
+        }
+        else if (a.startsWith("--base=")) {
+            base = a.slice("--base=".length);
+        }
         else if (a === "--full") {
             noTruncate = true;
         }
@@ -50,19 +57,19 @@ function parseArgs(args) {
             prUrl = a;
         }
     }
-    if (!prUrl) {
-        console.error("Usage: codiedev reverse-ticket <pr-url> [--with <artifact-key>] [--full]");
-        console.error("");
-        console.error("Example: codiedev reverse-ticket https://github.com/signalandcode/repo/pull/42");
-        process.exit(1);
-    }
-    return { prUrl, forcedKey, noTruncate };
+    return { prUrl, base, forcedKey, noTruncate };
 }
 async function runReverseTicket(args) {
-    const { prUrl, forcedKey, noTruncate } = parseArgs(args);
+    const { prUrl, base, forcedKey, noTruncate } = parseArgs(args);
+    // No PR URL → branch mode (mid-session, no PR exists yet).
+    if (!prUrl) {
+        return runBranchMode({ explicitBase: base, forcedKey });
+    }
     const parsed = parsePrUrl(prUrl);
     if (!parsed) {
         console.error("Couldn't parse PR URL. Expected format: https://github.com/<owner>/<repo>/pull/<number>");
+        console.error("");
+        console.error("Run with no arguments for branch mode (uses your current local branch).");
         process.exit(1);
     }
     if (!checkGhCli()) {
@@ -148,3 +155,135 @@ async function runReverseTicket(args) {
         process.exit(1);
     }
 }
+function git(cmd) {
+    try {
+        return (0, child_process_1.execSync)(`git ${cmd}`, {
+            stdio: ["pipe", "pipe", "pipe"],
+            maxBuffer: 32 * 1024 * 1024,
+        })
+            .toString("utf8")
+            .trim();
+    }
+    catch (err) {
+        throw new Error(`git ${cmd} failed: ${err.message}`);
+    }
+}
+function safeGit(fn) {
+    try {
+        return fn() || undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+async function runBranchMode(opts) {
+    // Confirm we're in a git repo.
+    try {
+        git("rev-parse --is-inside-work-tree");
+    }
+    catch {
+        console.error("Not inside a git repository. Run this from a checked-out repo.");
+        process.exit(1);
+    }
+    // Resolve base ref — explicit, then origin/main, then origin/master.
+    const candidates = opts.explicitBase
+        ? [opts.explicitBase]
+        : ["origin/main", "origin/master"];
+    let baseRef = null;
+    for (const c of candidates) {
+        try {
+            git(`rev-parse --verify ${c}`);
+            baseRef = c;
+            break;
+        }
+        catch {
+            // try next
+        }
+    }
+    if (!baseRef) {
+        console.error(`Couldn't find a base ref (tried ${candidates.join(", ")}). Pass --base <ref>.`);
+        process.exit(1);
+    }
+    const branch = git("rev-parse --abbrev-ref HEAD");
+    if (branch === "HEAD") {
+        console.error("Detached HEAD — checkout a branch first.");
+        process.exit(1);
+    }
+    const baseSha = git(`merge-base HEAD ${baseRef}`);
+    const headSha = git("rev-parse HEAD");
+    if (baseSha === headSha) {
+        console.error(`No commits on ${branch} ahead of ${baseRef} — make a commit first.`);
+        process.exit(1);
+    }
+    const filesRaw = git(`diff --name-only ${baseSha}...HEAD`);
+    const filesChanged = filesRaw ? filesRaw.split("\n").filter(Boolean) : [];
+    if (filesChanged.length === 0) {
+        console.error(`No files changed between ${baseRef} and ${branch} — nothing to write up.`);
+        process.exit(1);
+    }
+    let diff = "";
+    try {
+        diff = git(`diff ${baseSha}...HEAD`).slice(0, 12_000);
+    }
+    catch {
+        // best effort
+    }
+    let repo;
+    try {
+        const remoteUrl = git("config --get remote.origin.url");
+        const m = remoteUrl.match(/[:/]([^/:]+)\/([^/:]+?)(?:\.git)?$/);
+        if (!m) {
+            throw new Error(`Couldn't parse owner/repo from remote: ${remoteUrl}`);
+        }
+        repo = `${m[1]}/${m[2]}`;
+    }
+    catch (err) {
+        console.error(`Couldn't read git remote.origin.url: ${err.message}`);
+        process.exit(1);
+    }
+    const title = safeGit(() => git("log -1 --format=%s")) ?? `WIP: ${branch}`;
+    const author = safeGit(() => git("config --get user.name"));
+    const authorEmail = safeGit(() => git("config --get user.email"));
+    console.log(`Branch:  ${branch}  (${repo})`);
+    console.log(`Base:    ${baseRef} @ ${baseSha.slice(0, 12)}`);
+    console.log(`Head:    ${headSha.slice(0, 12)}`);
+    console.log(`Files:   ${filesChanged.length}`);
+    console.log("");
+    console.log("Generating ticket draft…");
+    console.log("");
+    const config = (0, shared_1.requireConfig)();
+    try {
+        const res = await (0, shared_1.apiRequest)("POST", "/api/cli/reverseTicketFromBranch", {
+            config,
+            body: {
+                branch: {
+                    repo,
+                    branch,
+                    baseSha,
+                    headSha,
+                    title,
+                    author,
+                    authorEmail,
+                    filesChanged,
+                    diff: diff || undefined,
+                },
+                forcedArtifactKey: opts.forcedKey,
+            },
+        });
+        if (!res.ok || !res.artifactId) {
+            console.error("Reverse-ticket generation returned no result. The writer may have skipped due to missing context.");
+            process.exit(1);
+        }
+        const verb = res.action === "overwritten" ? "Updated" : "Created";
+        console.log(`✓ ${verb} reverse-ticket draft.`);
+        console.log(`  artifact: ${res.artifactKey ?? res.artifactId}`);
+        console.log(`  portal:   ${res.portalUrl}`);
+        console.log("");
+        console.log("Open in your portal to review, edit, and push to Jira.");
+        console.log("When the PR opens later, the webhook dedupes against this draft — one ticket, not two.");
+    }
+    catch (err) {
+        console.error(`Reverse-ticket failed: ${err.message}`);
+        process.exit(1);
+    }
+}

package/dist/mcp.js

@@ -52,7 +52,7 @@ const path = __importStar(require("path"));
 const utils_1 = require("./utils");
 const shared_1 = require("./commands/shared");
 const PKG_NAME = "codiedev";
-const PKG_VERSION = "0.4.0";
+const PKG_VERSION = "0.5.0";
 // ─────────────────────────────────────────────────────────────────────────────
 // Tool definitions — descriptions tuned so Claude/Codex resolve natural-language
 // requests into the right tool without manual steering.
@@ -372,6 +372,41 @@ const TOOLS = [
             },
         },
     },
+    {
+        name: "codiedev_reverse_ticket",
+        description: "Generate a Jira-ready reverse-ticket draft from the work the user has " +
+            "done in their CURRENT git branch — mid-session, before any PR exists. " +
+            "Reads local git state (current branch, divergence from main, diff, " +
+            "commits, files changed) and runs it through the codiedev reverse-" +
+            "ticket writer. The draft lands in the user's portal at " +
+            "/portal/agentic-ticketing where they can edit the title, description, " +
+            "and scope, then push to Jira. Use when the user asks to 'create a " +
+            "reverse ticket', 'draft a Jira ticket from this work', 'write up what " +
+            "I've done in this branch', or runs `codiedev reverse-ticket` directly. " +
+            "The webhook on PR-open later dedupes against this draft so the user " +
+            "never gets a double ticket. The draft is intentionally not auto-pushed " +
+            "to Jira — trust comes from the user reviewing in the portal first.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                title: {
+                    type: "string",
+                    description: "Optional working title for the ticket. Defaults to the most " +
+                        "recent commit subject on the branch.",
+                },
+                base: {
+                    type: "string",
+                    description: "Optional base ref to diff against. Defaults to 'origin/main', " +
+                        "falls back to 'origin/master'.",
+                },
+                forcedArtifactKey: {
+                    type: "string",
+                    description: "Optional — force the writer to link to a specific artifact key " +
+                        "for context (e.g. 'spec-auth.md').",
+                },
+            },
+        },
+    },
 ];
 // ─────────────────────────────────────────────────────────────────────────────
 // Server
@@ -380,6 +415,26 @@ const server = new index_js_1.Server({ name: PKG_NAME, version: PKG_VERSION }, {
 server.setRequestHandler(types_js_1.ListToolsRequestSchema, async () => ({
     tools: TOOLS,
 }));
+/**
+ * Fire-and-forget telemetry — POST one event per tool invocation. Never blocks,
+ * never throws into the user-facing path. Drop silently on any failure.
+ */
+function emitTelemetry(config, event) {
+    if (!config)
+        return;
+    (0, shared_1.apiRequest)("POST", "/api/telemetry/mcp-event", {
+        config,
+        body: {
+            tool: event.tool,
+            ok: event.ok,
+            latencyMs: event.latencyMs,
+            error: event.error,
+            mcpVersion: PKG_VERSION,
+        },
+    }).catch(() => {
+        // Swallow — telemetry should never affect the user's session.
+    });
+}
 server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
     const config = (0, utils_1.readConfig)();
     if (!config) {
@@ -397,40 +452,26 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
     }
     const { name, arguments: argsRaw } = request.params;
     const args = (argsRaw ?? {});
+    const start = Date.now();
     try {
-        switch (name) {
-            case "codiedev_push":
-                return await handlePush(args, config);
-            case "codiedev_pull":
-                return await handlePull(args, config);
-            case "codiedev_ping":
-                return await handlePing(args, config);
-            case "codiedev_inbox":
-                return await handleInbox(args, config);
-            case "codiedev_note":
-                return await handleNote(args, config);
-            case "codiedev_promote":
-                return await handlePromote(args, config);
-            case "codiedev_post_to_feed":
-                return await handlePostToFeed(args, config);
-            case "codiedev_share_with":
-                return await handleShareWith(args, config);
-            case "codiedev_send_to":
-                return await handleSendTo(args, config);
-            case "codiedev_react":
-                return await handleReact(args, config);
-            case "codiedev_search":
-                return await handleSearch(args, config);
-            case "codiedev_get_library":
-                return await handleGetLibrary(args, config);
-            default:
-                return {
-                    isError: true,
-                    content: [{ type: "text", text: `Unknown tool: ${name}` }],
-                };
-        }
+        const result = await dispatchTool(name, args, config);
+        emitTelemetry(config, {
+            tool: name,
+            ok: !result?.isError,
+            latencyMs: Date.now() - start,
+            error: result?.isError
+                ? (result?.content?.[0]?.text ?? "").slice(0, 200) || "unknown error"
+                : undefined,
+        });
+        return result;
     }
     catch (err) {
+        emitTelemetry(config, {
+            tool: name,
+            ok: false,
+            latencyMs: Date.now() - start,
+            error: (err.message ?? "unknown").slice(0, 200),
+        });
         return {
             isError: true,
             content: [
@@ -442,6 +483,41 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
         };
     }
 });
+async function dispatchTool(name, args, config) {
+    switch (name) {
+        case "codiedev_push":
+            return await handlePush(args, config);
+        case "codiedev_pull":
+            return await handlePull(args, config);
+        case "codiedev_ping":
+            return await handlePing(args, config);
+        case "codiedev_inbox":
+            return await handleInbox(args, config);
+        case "codiedev_note":
+            return await handleNote(args, config);
+        case "codiedev_promote":
+            return await handlePromote(args, config);
+        case "codiedev_post_to_feed":
+            return await handlePostToFeed(args, config);
+        case "codiedev_share_with":
+            return await handleShareWith(args, config);
+        case "codiedev_send_to":
+            return await handleSendTo(args, config);
+        case "codiedev_react":
+            return await handleReact(args, config);
+        case "codiedev_search":
+            return await handleSearch(args, config);
+        case "codiedev_get_library":
+            return await handleGetLibrary(args, config);
+        case "codiedev_reverse_ticket":
+            return await handleReverseTicket(args, config);
+        default:
+            return {
+                isError: true,
+                content: [{ type: "text", text: `Unknown tool: ${name}` }],
+            };
+    }
+}
 async function handlePush(args, config) {
     const filename = asString(args.filename);
     const markdown = asString(args.markdown);
@@ -763,6 +839,153 @@ function formatAmbiguousRecipient(err, to) {
     }
     throw err;
 }
+async function handleReverseTicket(args, config) {
+    const explicitTitle = asStringOrUndefined(args.title);
+    const explicitBase = asStringOrUndefined(args.base);
+    const forcedKey = asStringOrUndefined(args.forcedArtifactKey);
+    // Helper that runs git from cwd with a hard-fail on error.
+    const { execSync } = await Promise.resolve().then(() => __importStar(require("child_process")));
+    const git = (cmd) => {
+        try {
+            return execSync(`git ${cmd}`, {
+                stdio: ["pipe", "pipe", "pipe"],
+                maxBuffer: 32 * 1024 * 1024,
+            })
+                .toString("utf8")
+                .trim();
+        }
+        catch (err) {
+            throw new Error(`git ${cmd} failed: ${err.message}`);
+        }
+    };
+    // Confirm we're in a git repo.
+    try {
+        git("rev-parse --is-inside-work-tree");
+    }
+    catch {
+        throw new Error("Not inside a git repository. Run this from a checked-out repo.");
+    }
+    // Resolve base ref — explicit, then origin/main, then origin/master.
+    const candidates = explicitBase
+        ? [explicitBase]
+        : ["origin/main", "origin/master"];
+    let baseRef = null;
+    for (const c of candidates) {
+        try {
+            git(`rev-parse --verify ${c}`);
+            baseRef = c;
+            break;
+        }
+        catch {
+            // try next
+        }
+    }
+    if (!baseRef) {
+        throw new Error(`Couldn't find a base ref (tried ${candidates.join(", ")}). ` +
+            `Pass an explicit base via the 'base' argument.`);
+    }
+    const branch = git("rev-parse --abbrev-ref HEAD");
+    if (branch === "HEAD") {
+        throw new Error("Detached HEAD — checkout a branch first.");
+    }
+    const baseSha = git(`merge-base HEAD ${baseRef}`);
+    const headSha = git("rev-parse HEAD");
+    if (baseSha === headSha) {
+        throw new Error(`No commits on ${branch} ahead of ${baseRef} — make a commit first.`);
+    }
+    const filesRaw = git(`diff --name-only ${baseSha}...HEAD`);
+    const filesChanged = filesRaw ? filesRaw.split("\n").filter(Boolean) : [];
+    if (filesChanged.length === 0) {
+        throw new Error(`No files changed between ${baseRef} and ${branch} — nothing to write up.`);
+    }
+    // Truncated diff (12k chars) — writer works without it but it sharpens output.
+    let diff = "";
+    try {
+        diff = git(`diff ${baseSha}...HEAD`).slice(0, 12_000);
+    }
+    catch {
+        // Best effort.
+    }
+    // Repo name from origin remote.
+    let repo;
+    try {
+        const remoteUrl = git("config --get remote.origin.url");
+        const m = remoteUrl.match(/[:/]([^/:]+)\/([^/:]+?)(?:\.git)?$/);
+        if (!m) {
+            throw new Error(`Couldn't parse owner/repo from remote: ${remoteUrl}`);
+        }
+        repo = `${m[1]}/${m[2]}`;
+    }
+    catch (err) {
+        throw new Error(`Couldn't read git remote.origin.url: ${err.message}`);
+    }
+    const title = explicitTitle ?? safeGit(() => git("log -1 --format=%s")) ?? `WIP: ${branch}`;
+    const author = safeGit(() => git("config --get user.name"));
+    const authorEmail = safeGit(() => git("config --get user.email"));
+    // Post to backend.
+    const res = await (0, shared_1.apiRequest)("POST", "/api/cli/reverseTicketFromBranch", {
+        config,
+        body: {
+            branch: {
+                repo,
+                branch,
+                baseSha,
+                headSha,
+                title,
+                author,
+                authorEmail,
+                filesChanged,
+                diff: diff || undefined,
+            },
+            forcedArtifactKey: forcedKey,
+        },
+    });
+    if (!res.ok || !res.artifactId) {
+        return {
+            isError: true,
+            content: [
+                {
+                    type: "text",
+                    text: "Reverse-ticket generation returned no result. The writer may have " +
+                        "skipped due to missing context (no captured sessions, etc.). Try " +
+                        "again after working on the branch a bit more, or check the " +
+                        "server logs.",
+                },
+            ],
+        };
+    }
+    const portalBase = (config.backendUrl || "")
+        .replace(/\/$/, "")
+        .replace(/\.convex\.cloud$/, ".convex.cloud"); // backendUrl is the Convex site; portal is separate
+    // Best effort — surface the portal path; the user can prefix with their portal host.
+    const portalPath = res.portalUrl;
+    const verb = res.action === "overwritten" ? "Updated" : "Created";
+    return {
+        content: [
+            {
+                type: "text",
+                text: `✓ ${verb} reverse-ticket draft for branch \`${branch}\` (${repo}).\n` +
+                    `  files changed: ${filesChanged.length}\n` +
+                    `  base: ${baseSha.slice(0, 12)} → head: ${headSha.slice(0, 12)} (${baseRef})\n` +
+                    `  diff size: ${diff.length} chars\n` +
+                    `  artifact: ${res.artifactKey ?? res.artifactId}\n\n` +
+                    `Open in your portal to review, edit, and push to Jira:\n` +
+                    `  ${portalPath}\n\n` +
+                    `When the PR opens later, the webhook dedupes against this draft — ` +
+                    `you'll get one ticket, not two.`,
+            },
+        ],
+    };
+}
+// Run a git invocation; return undefined instead of throwing.
+function safeGit(fn) {
+    try {
+        return fn();
+    }
+    catch {
+        return undefined;
+    }
+}
 // ─────────────────────────────────────────────────────────────────────────────
 // Helpers
 // ─────────────────────────────────────────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codiedev",
-  "version": "0.4.0",
+  "version": "0.5.1",
   "description": "Connect Claude Code or Codex to CodieDev for org-wide session capture and artifact collaboration",
   "bin": {
     "codiedev": "./dist/cli.js",