@shipfox/api-integration-github 2.0.0 → 4.0.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/api-integration-github",
   "license": "MIT",
-  "version": "2.0.0",
+  "version": "4.0.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -37,26 +37,26 @@
     "ky": "^2.0.0",
     "octokit": "^5.0.3",
     "zod": "^4.4.3",
-    "@shipfox/api-auth-context": "2.0.0",
-    "@shipfox/api-integration-core-dto": "2.0.0",
-    "@shipfox/api-workspaces": "2.0.0",
-    "@shipfox/api-integration-github-dto": "2.0.0",
+    "@shipfox/api-auth-context": "3.0.0",
+    "@shipfox/api-integration-core-dto": "3.0.0",
+    "@shipfox/api-integration-github-dto": "3.0.0",
+    "@shipfox/api-workspaces": "4.0.0",
     "@shipfox/config": "1.2.1",
-    "@shipfox/node-fastify": "0.2.1",
-    "@shipfox/node-drizzle": "0.2.1",
+    "@shipfox/node-drizzle": "0.3.0",
+    "@shipfox/node-fastify": "0.2.2",
     "@shipfox/node-postgres": "0.4.1",
-    "@shipfox/node-opentelemetry": "0.5.0"
+    "@shipfox/node-opentelemetry": "0.5.1"
   },
   "devDependencies": {
     "@types/pg": "^8.15.5",
     "drizzle-kit": "^0.31.10",
     "fastify": "^5.3.3",
     "fastify-type-provider-zod": "^6.0.0",
-    "fishery": "^2.2.2",
-    "@shipfox/typescript": "1.1.6",
-    "@shipfox/ts-config": "1.3.8",
+    "fishery": "^2.4.0",
     "@shipfox/biome": "1.8.1",
     "@shipfox/swc": "1.2.5",
+    "@shipfox/ts-config": "1.3.8",
+    "@shipfox/typescript": "1.1.6",
     "@shipfox/vitest": "1.2.2"
   },
   "scripts": {

package/src/core/actions-logs.ts

@@ -1,6 +1,7 @@
 import {Buffer} from 'node:buffer';
+import {DEFAULT_JOB_LOG_TAIL_LINES} from '@shipfox/api-integration-github-dto';
 
-export const DEFAULT_JOB_LOG_TAIL_LINES = 500;
+export {DEFAULT_JOB_LOG_TAIL_LINES} from '@shipfox/api-integration-github-dto';
 export const DEFAULT_JOB_LOG_CONTENT_WINDOW_LINES = 5000;
 export const MAX_JOB_LOG_RING_BUFFER_LINES = 100000;
 export const MAX_JOB_LOG_LINE_BYTES = 10 * 1024 * 1024;

package/src/core/agent-tool-selection.ts

@@ -1,42 +1 @@
-import type {AgentToolSelectionCatalog, AgentToolSelector} from '@shipfox/api-integration-core-dto';
-import type {GithubAgentToolCatalogEntry} from './agent-tools.js';
-
-export function buildGithubAgentToolSelectionCatalog(
-  catalog: readonly GithubAgentToolCatalogEntry[],
-): AgentToolSelectionCatalog {
-  return {
-    selectors: catalog.flatMap((entry): AgentToolSelector[] => {
-      if (!entry.methods) {
-        return [
-          {
-            token: entry.id,
-            kind: 'standalone',
-            sensitivity: entry.sensitivity,
-            sensitive: entry.sensitive,
-          },
-        ];
-      }
-
-      return [
-        {
-          token: entry.id,
-          kind: 'family',
-          sensitivity: entry.sensitivity,
-          sensitive: entry.sensitive,
-        },
-        {
-          token: `${entry.id}.*`,
-          kind: 'family_wildcard',
-          sensitivity: entry.sensitivity,
-          sensitive: entry.sensitive,
-        },
-        ...entry.methods.map((method) => ({
-          token: `${entry.id}.${method.id}`,
-          kind: 'method' as const,
-          sensitivity: method.sensitivity,
-          sensitive: method.sensitive,
-        })),
-      ];
-    }),
-  };
-}
+export {buildGithubAgentToolSelectionCatalog} from '@shipfox/api-integration-github-dto';

package/src/core/agent-tools.test.ts

@@ -140,7 +140,7 @@ const expectedCatalogRows = [
     sensitivity: 'write',
     sensitive: false,
     requiredScope: [{permission: 'pull_requests', access: 'write'}],
-    methods: ['create', 'submit_pending', 'delete_pending', 'resolve_thread', 'unresolve_thread'],
+    methods: ['create', 'submit_pending', 'delete_pending'],
   },
   {
     id: 'add_comment_to_pending_review',
@@ -259,6 +259,7 @@ describe('github agent tool catalog', () => {
     const addIssueCommentSchema = inputSchemaFor('add_issue_comment');
     const updatePullRequestSchema = inputSchemaFor('update_pull_request');
     const addReplySchema = inputSchemaFor('add_reply_to_pull_request_comment');
+    const pullRequestReadSchema = inputSchemaFor('pull_request_read');
     const actionsRunTriggerSchema = inputSchemaFor('actions_run_trigger');
     const getJobLogsSchema = inputSchemaFor('get_job_logs');
 
@@ -273,6 +274,17 @@ describe('github agent tool catalog', () => {
       {required: ['pull_number', 'body']},
       {required: ['reaction']},
     ]);
+    expect(pullRequestReadSchema.oneOf).toEqual([
+      {properties: {method: {const: 'get'}}, required: []},
+      {properties: {method: {const: 'get_diff'}}, required: []},
+      {properties: {method: {const: 'get_status'}}, required: ['ref']},
+      {properties: {method: {const: 'get_files'}}, required: []},
+      {properties: {method: {const: 'get_commits'}}, required: []},
+      {properties: {method: {const: 'get_review_comments'}}, required: []},
+      {properties: {method: {const: 'get_reviews'}}, required: []},
+      {properties: {method: {const: 'get_comments'}}, required: []},
+      {properties: {method: {const: 'get_check_runs'}}, required: ['ref']},
+    ]);
     expect(actionsRunTriggerSchema.oneOf).toEqual([
       {properties: {method: {const: 'run_workflow'}}, required: ['workflow_id', 'ref']},
       {properties: {method: {const: 'rerun_workflow_run'}}, required: ['run_id']},
@@ -357,6 +369,19 @@ describe('github agent tool catalog', () => {
     });
   });
 
+  it('requires a ref for pull request status and check-run reads', async () => {
+    const result = await callGithubTool(
+      'pull_request_read',
+      {method: 'get_status', owner: 'shipfox', repo: 'platform', pull_number: 2},
+      {state: 'success'},
+    );
+
+    expect(result).toEqual({
+      isError: true,
+      content: [{type: 'text', text: 'Missing required parameter: ref'}],
+    });
+  });
+
   it.each([
     {
       toolId: 'list_issue_types',
@@ -423,6 +448,18 @@ describe('github agent tool catalog', () => {
       data: 'diff --git a/file b/file',
       expected: {result: 'diff --git a/file b/file'},
     },
+    {
+      toolId: 'pull_request_read',
+      arguments: {
+        method: 'get_check_runs',
+        owner: 'shipfox',
+        repo: 'platform',
+        pull_number: 2,
+        ref: 'abc123',
+      },
+      data: {total_count: 1},
+      expected: {total_count: 1},
+    },
   ] satisfies Array<{
     toolId: GithubAgentToolId;
     arguments: Record<string, unknown>;