@cliangdev/flux-plugin 0.3.1-dev.ee3b5ee → 0.4.0-dev.0892a21

package/src/server/tools/__tests__/z-configure-github.test.ts

@@ -26,8 +26,6 @@ let configCommitted = false;
 
 mock.module("@octokit/rest", () => ({
   Octokit: class MockOctokit {
-    constructor(_opts: { auth: string }) {}
-
     users = {
       getAuthenticated: async () => {
         if (!mockTokenValid) {
@@ -81,7 +79,7 @@ mock.module("@octokit/rest", () => ({
 
     repos_getContent = {};
 
-    request = async (route: string, opts: any) => {
+    request = async (route: string, _opts: any) => {
       if (route === "GET /repos/{owner}/{repo}/contents/{path}") {
         if (!mockRemoteConfigExists) {
           const err: any = new Error("Not Found");
@@ -109,7 +107,7 @@ mock.module("@octokit/graphql", () => ({
     },
     {
       defaults: (_opts: any) => {
-        return async (query: string, vars: any) => {
+        return async (query: string, _vars: any) => {
           if (query.includes("GetOwnerId")) {
             return { user: { id: mockOwnerId } };
           }
@@ -120,6 +118,14 @@ mock.module("@octokit/graphql", () => ({
               },
             };
           }
+          if (query.includes("GetRepoId")) {
+            return { repository: { id: "REPO_NODE_ID" } };
+          }
+          if (query.includes("LinkProject")) {
+            return {
+              linkProjectV2ToRepository: { repository: { id: "REPO_NODE_ID" } },
+            };
+          }
           throw new Error(`Unhandled GraphQL query: ${query}`);
         };
       },
@@ -170,12 +176,18 @@ describe("configure_github MCP Tool", () => {
 
   describe("input validation", () => {
     test("rejects missing token", async () => {
-      await expect(
-        configureGithubTool.handler({
-          owner: "testowner",
-          repo: "flux-tracking",
-        }),
-      ).rejects.toThrow();
+      const saved = process.env.FLUX_GITHUB_TOKEN;
+      delete process.env.FLUX_GITHUB_TOKEN;
+      try {
+        await expect(
+          configureGithubTool.handler({
+            owner: "testowner",
+            repo: "flux-tracking",
+          }),
+        ).rejects.toThrow();
+      } finally {
+        if (saved !== undefined) process.env.FLUX_GITHUB_TOKEN = saved;
+      }
     });
 
     test("rejects missing owner", async () => {

package/src/server/tools/__tests__/z-get-linear-url.test.ts

@@ -49,7 +49,7 @@ describe("get_linear_url MCP Tool", () => {
   test("returns URL for project ref", async () => {
     // Get the adapter and mock its getLinearUrl method
     const adapter = getAdapter() as any;
-    adapter.getLinearUrl = mock(async (ref: string) => ({
+    adapter.getLinearUrl = mock(async (_ref: string) => ({
       url: "https://linear.app/myteam/project/my-project-abc123",
       type: "project",
       name: "My Project",
@@ -69,7 +69,7 @@ describe("get_linear_url MCP Tool", () => {
   test("returns URL for issue identifier", async () => {
     // Get the adapter and mock its getLinearUrl method
     const adapter = getAdapter() as any;
-    adapter.getLinearUrl = mock(async (ref: string) => ({
+    adapter.getLinearUrl = mock(async (_ref: string) => ({
       url: "https://linear.app/myteam/issue/ENG-42",
       type: "issue",
       identifier: "ENG-42",

package/src/server/tools/configure-github.ts

@@ -51,7 +51,8 @@ async function validateToken(octokit: Octokit): Promise<string> {
   try {
     const { data } = await octokit.users.getAuthenticated();
     return data.login;
-  } catch (err: any) {
+  } catch (error: unknown) {
+    const err = error as { status?: number; message?: string };
     const status = err.status ?? 0;
     throw new Error(
       `GitHub token validation failed (HTTP ${status}): ${err.message}. Ensure the token has 'repo' scope.`,
@@ -90,11 +91,12 @@ async function fetchRemoteConfig(
     const data = response.data as { content: string; encoding: string };
     const decoded = Buffer.from(data.content, "base64").toString("utf-8");
     return JSON.parse(decoded) as SharedConfig;
-  } catch (err: any) {
-    if (err.status === 404 || err.status === 403) {
+  } catch (error: unknown) {
+    const status = (error as { status?: number }).status;
+    if (status === 404 || status === 403) {
       return null;
     }
-    throw err;
+    throw error;
   }
 }
 
@@ -124,8 +126,8 @@ async function ensureRepo(
   try {
     const { data } = await octokit.repos.get({ owner, repo });
     return data.html_url;
-  } catch (err: any) {
-    if (err.status !== 404) throw err;
+  } catch (error: unknown) {
+    if ((error as { status?: number }).status !== 404) throw error;
   }
 
   const { data } = await octokit.repos.createForAuthenticatedUser({
@@ -163,9 +165,14 @@ async function ensureLabels(
 }
 
 async function createProjectsBoard(
-  gql: (query: string, vars: any) => Promise<any>,
+  gql: (
+    query: string,
+    vars: Record<string, unknown>,
+  ) => Promise<Record<string, Record<string, unknown>>>,
   ownerLogin: string,
   title: string,
+  repoOwner: string,
+  repoName: string,
 ): Promise<{ id: string; url: string }> {
   const ownerResult = await gql(
     `query GetOwnerId($login: String!) { user(login: $login) { id } }`,
@@ -182,7 +189,29 @@ async function createProjectsBoard(
     { ownerId, title },
   );
 
-  return projectResult.createProjectV2.projectV2;
+  const project = projectResult.createProjectV2.projectV2 as unknown as {
+    id: string;
+    url: string;
+  };
+
+  const repoResult = await gql(
+    `query GetRepoId($owner: String!, $name: String!) {
+      repository(owner: $owner, name: $name) { id }
+    }`,
+    { owner: repoOwner, name: repoName },
+  );
+  const repositoryId = repoResult.repository.id;
+
+  await gql(
+    `mutation LinkProject($projectId: ID!, $repositoryId: ID!) {
+      linkProjectV2ToRepository(input: { projectId: $projectId, repositoryId: $repositoryId }) {
+        repository { id }
+      }
+    }`,
+    { projectId: project.id, repositoryId },
+  );
+
+  return project;
 }
 
 async function commitSharedConfig(
@@ -231,7 +260,10 @@ function writeProjectJson(params: {
 
 async function runSetup(params: {
   octokit: Octokit;
-  gql: (query: string, vars: any) => Promise<any>;
+  gql: (
+    query: string,
+    vars: Record<string, unknown>,
+  ) => Promise<Record<string, Record<string, unknown>>>;
   ownerLogin: string;
   owner: string;
   repo: string;
@@ -254,6 +286,8 @@ async function runSetup(params: {
     params.gql,
     params.ownerLogin,
     `Flux: ${params.repo}`,
+    params.owner,
+    params.repo,
   );
 
   const sharedConfig: SharedConfig = {