@craftsamo/agent-github 0.1.0 → 0.2.0

package/dist/mcp.js

@@ -3911,50 +3911,50 @@ var require_fast_uri = __commonJS({
       schemelessOptions.skipEscape = true;
       return serialize(resolved, schemelessOptions);
     }
-    function resolveComponent(base, relative, options, skipNormalization) {
+    function resolveComponent(base, relative2, options, skipNormalization) {
       const target = {};
       if (!skipNormalization) {
         base = parse3(serialize(base, options), options);
-        relative = parse3(serialize(relative, options), options);
+        relative2 = parse3(serialize(relative2, options), options);
       }
       options = options || {};
-      if (!options.tolerant && relative.scheme) {
-        target.scheme = relative.scheme;
-        target.userinfo = relative.userinfo;
-        target.host = relative.host;
-        target.port = relative.port;
-        target.path = removeDotSegments(relative.path || "");
-        target.query = relative.query;
+      if (!options.tolerant && relative2.scheme) {
+        target.scheme = relative2.scheme;
+        target.userinfo = relative2.userinfo;
+        target.host = relative2.host;
+        target.port = relative2.port;
+        target.path = removeDotSegments(relative2.path || "");
+        target.query = relative2.query;
       } else {
-        if (relative.userinfo !== void 0 || relative.host !== void 0 || relative.port !== void 0) {
-          target.userinfo = relative.userinfo;
-          target.host = relative.host;
-          target.port = relative.port;
-          target.path = removeDotSegments(relative.path || "");
-          target.query = relative.query;
+        if (relative2.userinfo !== void 0 || relative2.host !== void 0 || relative2.port !== void 0) {
+          target.userinfo = relative2.userinfo;
+          target.host = relative2.host;
+          target.port = relative2.port;
+          target.path = removeDotSegments(relative2.path || "");
+          target.query = relative2.query;
         } else {
-          if (!relative.path) {
+          if (!relative2.path) {
             target.path = base.path;
-            if (relative.query !== void 0) {
-              target.query = relative.query;
+            if (relative2.query !== void 0) {
+              target.query = relative2.query;
             } else {
               target.query = base.query;
             }
           } else {
-            if (relative.path[0] === "/") {
-              target.path = removeDotSegments(relative.path);
+            if (relative2.path[0] === "/") {
+              target.path = removeDotSegments(relative2.path);
             } else {
               const baseHasAuthority = base.userinfo !== void 0 || base.host !== void 0 || base.port !== void 0;
               if (baseHasAuthority && !base.path) {
-                target.path = "/" + relative.path;
+                target.path = "/" + relative2.path;
               } else if (!base.path) {
-                target.path = relative.path;
+                target.path = relative2.path;
               } else {
-                target.path = base.path.slice(0, base.path.lastIndexOf("/") + 1) + relative.path;
+                target.path = base.path.slice(0, base.path.lastIndexOf("/") + 1) + relative2.path;
               }
               target.path = base.scheme === void 0 && !baseHasAuthority && target.path[0] !== "/" ? removeDotSegmentsRelative(target.path) : removeDotSegments(target.path);
             }
-            target.query = relative.query;
+            target.query = relative2.query;
           }
           target.userinfo = base.userinfo;
           target.host = base.host;
@@ -3962,7 +3962,7 @@ var require_fast_uri = __commonJS({
         }
         target.scheme = base.scheme;
       }
-      target.fragment = relative.fragment;
+      target.fragment = relative2.fragment;
       return target;
     }
     function equal(uriA, uriB, options) {
@@ -28956,6 +28956,8 @@ async function paginate(query, vars, pick2, options) {
 }
 
 // src/core/refs.ts
+import { realpath } from "node:fs/promises";
+import { isAbsolute, relative, sep } from "node:path";
 var DEFAULT_HOST = "github.com";
 var SSH_ALIAS_HOST = "ssh.github.com";
 var GIT_CONFIG_ENV = ["GIT_CONFIG_GLOBAL", "GIT_CONFIG_SYSTEM", "GIT_CONFIG_NOSYSTEM"];
@@ -29039,11 +29041,25 @@ function gitEnv() {
   }
   return { ...env, GIT_TERMINAL_PROMPT: "0" };
 }
-async function repoFromOrigin(cwd, signal) {
+async function isWithin(dir, root) {
+  try {
+    const path = relative(await realpath(root), await realpath(dir));
+    return path === "" || !(path === ".." || path.startsWith(`..${sep}`) || isAbsolute(path));
+  } catch {
+    return false;
+  }
+}
+async function repoFromOrigin(cwd, signal, ownDir) {
   const env = gitEnv();
-  const unresolved = (why) => invalid(
-    `Cannot tell which repository to use: ${why}. Run the call inside a clone of a GitHub repository, or pass \`repo\` as owner/name.`
+  const unresolved = (why, advice = "") => invalid(
+    `Cannot tell which repository to use: ${why}. Run the call inside a clone of a GitHub repository, or pass \`repo\` as owner/name.${advice}`
   );
+  if (ownDir !== void 0 && await isWithin(cwd, ownDir)) {
+    throw unresolved(
+      `${cwd} is where these tools are installed, not a project`,
+      " An issue can be given as `owner/repo#N`."
+    );
+  }
   let result;
   try {
     result = await exec(["git", "remote", "get-url", "origin"], { cwd, env, signal });
@@ -29058,7 +29074,10 @@ async function repoFromOrigin(cwd, signal) {
   return repo;
 }
 async function resolveRepo(repo, ctx) {
-  return repo === void 0 || repo.trim() === "" ? repoFromOrigin(ctx.cwd, ctx.signal) : parseRepo(repo);
+  return repo === void 0 || repo.trim() === "" ? repoFromOrigin(ctx.cwd, ctx.signal, ctx.ownDir) : parseRepo(repo);
+}
+function isBareNumber(ref) {
+  return BARE_NUMBER_PATTERN.test(ref.trim());
 }
 function parseNumber(digits, ref) {
   const number4 = Number.parseInt(digits, 10);
@@ -29128,6 +29147,10 @@ function defineTool(spec) {
   if (!(spec.output instanceof external_exports.ZodObject)) {
     throw new Error(`Tool "${spec.id}" must have an object output schema, since a result is an object.`);
   }
+  const unknownTarget = Object.keys(spec.targets ?? {}).find((name) => !Object.hasOwn(spec.input.shape, name));
+  if (unknownTarget !== void 0) {
+    throw new Error(`Tool "${spec.id}" declares the target "${unknownTarget}", which is not one of its arguments.`);
+  }
   return spec;
 }
 var CLASS_HINTS = {
@@ -29233,6 +29256,13 @@ var StepErrors = class {
   }
 };
 
+// src/entry/package-root.ts
+import { dirname } from "node:path";
+import { fileURLToPath } from "node:url";
+function packageRoot(bundleUrl) {
+  return dirname(dirname(fileURLToPath(bundleUrl)));
+}
+
 // src/entry/settings.ts
 var CONFIRM_MODES = ["harness", "elicitation"];
 var ALL_TOOLSETS = "all";
@@ -29758,10 +29788,9 @@ async function readSubIssues(parent, ctx) {
 }
 
 // src/core/github/issue-refs.ts
-var BARE_NUMBER = /^#?\d+$/;
 async function resolveRelatedIssue(ref, target, ctx) {
   const text = ref.trim();
-  return parseNumberRef(text, BARE_NUMBER.test(text) ? target.full : void 0, ctx, { accept: "issue" });
+  return parseNumberRef(text, isBareNumber(text) ? target.full : void 0, ctx, { accept: "issue" });
 }
 function issueLabel(issue2) {
   return `${issue2.repo.full}#${issue2.number}`;
@@ -29773,6 +29802,7 @@ function ghIssueRef(issue2, target) {
 // src/core/issues/write-support.ts
 var REPO_DESCRIPTION = "owner/name; defaults to the working tree's repository (its origin). Used for a bare number.";
 var ISSUE_DESCRIPTION = "The issue, as 12, #12, owner/repo#12 or its URL.";
+var ISSUE_TARGETS = { issue: "issue", repo: "repo" };
 var LIST_SYNTAX = /[,"\r\n]/;
 var URL_LINE = /^https?:\/\/\S+$/;
 function invalid2(message) {
@@ -29842,6 +29872,7 @@ var issueClose = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "overwrite",
+  targets: ISSUE_TARGETS,
   input: input2,
   output: output2,
   async execute(args, ctx) {
@@ -29898,6 +29929,7 @@ var issueComment = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "add",
+  targets: ISSUE_TARGETS,
   input: input3,
   output: output3,
   async execute(args, ctx) {
@@ -30015,6 +30047,7 @@ var issueCreate = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "add",
+  targets: { repo: "repo", parent: "issue", blocked_by: "issues" },
   input: input4,
   output: output4,
   async execute(args, ctx) {
@@ -30162,6 +30195,7 @@ var issueDevelop = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "idempotent",
+  targets: { ...ISSUE_TARGETS, branch_repo: "repo" },
   input: input5,
   output: output5,
   async execute(args, ctx) {
@@ -30309,6 +30343,7 @@ var issueEdit = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "overwrite",
+  targets: ISSUE_TARGETS,
   input: input6,
   output: output6,
   async execute(args, ctx) {
@@ -30667,6 +30702,16 @@ var issueRelate = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "idempotent",
+  targets: {
+    ...ISSUE_TARGETS,
+    parent: "issue",
+    add_sub_issues: "issues",
+    remove_sub_issues: "issues",
+    add_blocked_by: "issues",
+    remove_blocked_by: "issues",
+    add_blocking: "issues",
+    remove_blocking: "issues"
+  },
   input: input8,
   output: output8,
   async execute(args, ctx) {
@@ -30765,6 +30810,7 @@ var issueReopen = defineTool({
   ].join(" "),
   toolset: "issues",
   class: "overwrite",
+  targets: ISSUE_TARGETS,
   input: input9,
   output: output9,
   async execute(args, ctx) {
@@ -32582,7 +32628,7 @@ function elicitationConfirm(server, signal) {
 
 // src/mcp/roots.ts
 import { statSync } from "node:fs";
-import { fileURLToPath } from "node:url";
+import { fileURLToPath as fileURLToPath2 } from "node:url";
 var FILE_URI_PROTOCOL = "file:";
 async function isInsideWorkTree(dir, signal) {
   try {
@@ -32597,21 +32643,22 @@ function localDirectory(uri) {
   try {
     const url2 = new URL(uri);
     if (url2.protocol !== FILE_URI_PROTOCOL) return void 0;
-    const dir = fileURLToPath(url2);
+    const dir = fileURLToPath2(url2);
     return statSync(dir, { throwIfNoEntry: false })?.isDirectory() ? dir : void 0;
   } catch {
     return void 0;
   }
 }
-async function firstGitRoot(server, signal) {
+async function firstGitRoot(server, ownDir, signal) {
   const { roots } = await server.listRoots(void 0, { signal });
   for (const root of roots) {
     const dir = localDirectory(root.uri);
-    if (dir !== void 0 && await isInsideWorkTree(dir, signal)) return dir;
+    if (dir === void 0 || ownDir !== void 0 && await isWithin(dir, ownDir)) continue;
+    if (await isInsideWorkTree(dir, signal)) return dir;
   }
   return void 0;
 }
-function createWorkingDirResolver(server, fallback) {
+function createWorkingDirResolver(server, fallback, ownDir) {
   let cached2;
   let generation = 0;
   server.setNotificationHandler(RootsListChangedNotificationSchema, () => {
@@ -32624,7 +32671,7 @@ function createWorkingDirResolver(server, fallback) {
     const started = generation;
     let dir;
     try {
-      dir = await firstGitRoot(server, signal) ?? fallback();
+      dir = await firstGitRoot(server, ownDir, signal) ?? fallback();
     } catch (error62) {
       if (signal?.aborted || isAbortError(error62)) throw error62;
       return fallback();
@@ -32659,7 +32706,7 @@ function createServer(specs, settings, options = {}) {
     { name: SERVER_NAME, version: options.version ?? DEFAULT_VERSION },
     { capabilities: { tools: {} } }
   );
-  const workingDir = createWorkingDirResolver(server, options.cwd ?? (() => process.cwd()));
+  const workingDir = createWorkingDirResolver(server, options.cwd ?? (() => process.cwd()), options.ownDir);
   const selected = new Map(selectTools(specs, settings).map((spec) => [spec.id, spec]));
   const tools = [...selected.values()].map(describeTool);
   server.setRequestHandler(ListToolsRequestSchema, () => ({ tools }));
@@ -32670,6 +32717,7 @@ function createServer(specs, settings, options = {}) {
       const confirm = settings.confirm === "elicitation" ? elicitationConfirm(server, extra.signal) : void 0;
       const ctx = {
         cwd: await workingDir(extra.signal),
+        ownDir: options.ownDir,
         signal: extra.signal,
         ...confirm && { confirm }
       };
@@ -32694,7 +32742,8 @@ async function main() {
   try {
     const settings = parseSettings(process.argv.slice(2), process.env);
     server = createServer(ALL_TOOLS, settings, {
-      version: true ? "0.1.0" : void 0
+      ownDir: packageRoot(import.meta.url),
+      version: true ? "0.2.0" : void 0
     });
   } catch (error62) {
     if (!isToolError(error62)) throw error62;