nugit-cli 0.0.1 → 0.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nugit-cli",
-  "version": "0.0.1",
+  "version": "0.1.1",
   "description": "CLI for stacked GitHub PRs (.nugit/stack.json)",
   "type": "module",
   "bin": {

package/src/api-client.js

@@ -7,6 +7,7 @@ import {
   githubListUserRepos,
   githubRestJson,
   githubSearchIssues,
+  githubSearchRepositories,
   githubListOpenPulls
 } from "./github-rest.js";
 import {
@@ -14,6 +15,7 @@ import {
   githubDeviceFlowRequestCode
 } from "./github-device-flow.js";
 import { resolveGithubToken } from "./auth-token.js";
+import { resolveGithubOAuthClientId } from "./github-oauth-client-id.js";
 
 /** @returns {string} */
 export function getToken() {
@@ -29,15 +31,10 @@ export function withAuthHeaders(headers = {}) {
 }
 
 /**
- * Start GitHub OAuth device flow (requires GITHUB_OAUTH_CLIENT_ID).
+ * Start GitHub OAuth device flow (bundled OAuth App client id; override with GITHUB_OAUTH_CLIENT_ID).
  */
 export async function startDeviceFlow() {
-  const clientId = process.env.GITHUB_OAUTH_CLIENT_ID;
-  if (!clientId) {
-    throw new Error(
-      "Set GITHUB_OAUTH_CLIENT_ID to your GitHub OAuth App client ID (Settings → Developer settings → OAuth Apps), or use a PAT with NUGIT_USER_TOKEN."
-    );
-  }
+  const clientId = resolveGithubOAuthClientId();
   return githubDeviceFlowRequestCode(clientId, "repo read:user user:email");
 }
 
@@ -47,10 +44,7 @@ export async function startDeviceFlow() {
  * @param {number} [intervalSeconds] minimum wait before next poll (from GitHub or prior slow_down)
  */
 export async function pollDeviceFlow(deviceCode, intervalSeconds = 5) {
-  const clientId = process.env.GITHUB_OAUTH_CLIENT_ID;
-  if (!clientId) {
-    throw new Error("Set GITHUB_OAUTH_CLIENT_ID");
-  }
+  const clientId = resolveGithubOAuthClientId();
   const payload = await githubDeviceFlowPollAccessToken(clientId, deviceCode);
   if (payload.access_token) {
     return { access_token: payload.access_token, token_type: payload.token_type, scope: payload.scope };
@@ -145,6 +139,11 @@ export async function getRepoMetadata(owner, repo) {
   return githubGetRepoMetadata(owner, repo);
 }
 
+/** @param {string} query GitHub `q` syntax for /search/repositories */
+export async function searchRepositories(query, perPage = 20, page = 1) {
+  return githubSearchRepositories(query, perPage, page);
+}
+
 export async function createPullRequest(owner, repo, fields) {
   return githubCreatePullRequest(owner, repo, fields);
 }
@@ -160,18 +159,6 @@ export function decodeGithubFileContent(item) {
   return Buffer.from(item.content.replace(/\s/g, ""), "base64").toString("utf8");
 }
 
-export async function fetchRemoteStackJson(repoFullName, ref) {
-  const [owner, repo] = repoFullName.split("/");
-  if (!owner || !repo) {
-    throw new Error("repoFullName must be owner/repo");
-  }
-  const item = await getGithubContents(owner, repo, ".nugit/stack.json", ref);
-  const text = decodeGithubFileContent(item);
-  if (!text) {
-    throw new Error("Could not read .nugit/stack.json from GitHub");
-  }
-  return JSON.parse(text);
-}
 
 export async function getPull(owner, repo, number) {
   return githubGetPull(owner, repo, number);

package/src/github-device-flow.js

@@ -1,5 +1,5 @@
 /**
- * GitHub OAuth device flow (no backend). Requires GITHUB_OAUTH_CLIENT_ID.
+ * GitHub OAuth device flow (no backend). Client id from resolveGithubOAuthClientId() (bundled default or GITHUB_OAUTH_CLIENT_ID).
  * @see https://docs.github.com/en/apps/oauth-apps/building-oauth-apps/authorizing-oauth-apps#device-flow
  */
 

package/src/github-oauth-client-id.js

@@ -0,0 +1,11 @@
+/**
+ * Default GitHub OAuth App for `nugit auth login` (device flow).
+ * Client IDs are public (not secrets). Override with env for a custom app or fork.
+ */
+export const NUGIT_BUNDLED_GITHUB_OAUTH_CLIENT_ID = "Ov23liMBQqJJvRNiO0qm";
+
+/** @returns {string} */
+export function resolveGithubOAuthClientId() {
+  const fromEnv = process.env.GITHUB_OAUTH_CLIENT_ID?.trim();
+  return fromEnv || NUGIT_BUNDLED_GITHUB_OAUTH_CLIENT_ID;
+}

package/src/github-pr-social.js

@@ -124,3 +124,45 @@ export async function githubListAssignableUsers(owner, repo) {
   const r = encodeURIComponent(repo);
   return githubGetAllPages(`/repos/${o}/${r}/assignees`);
 }
+
+/**
+ * All pull reviews (paginated).
+ * @param {string} owner
+ * @param {string} repo
+ * @param {number} pullNumber
+ */
+export async function githubListPullReviewsAll(owner, repo, pullNumber) {
+  const o = encodeURIComponent(owner);
+  const r = encodeURIComponent(repo);
+  const n = encodeURIComponent(String(pullNumber));
+  return githubGetAllPages(`/repos/${o}/${r}/pulls/${n}/reviews`);
+}
+
+/**
+ * Post a single line-linked review comment on a PR (creates a new review thread).
+ *
+ * @param {string} owner
+ * @param {string} repo
+ * @param {number} pullNumber
+ * @param {{ body: string, commit_id: string, path: string, line: number, side?: "LEFT" | "RIGHT" }} payload
+ */
+export async function githubPostPullReviewLineComment(owner, repo, pullNumber, payload) {
+  const o = encodeURIComponent(owner);
+  const r = encodeURIComponent(repo);
+  const n = encodeURIComponent(String(pullNumber));
+  return githubRestJson("POST", `/repos/${o}/${r}/pulls/${n}/comments`, payload);
+}
+
+/**
+ * Submit a PR review (approve, comment, request changes).
+ * @param {string} owner
+ * @param {string} repo
+ * @param {number} pullNumber
+ * @param {{ event: string, body?: string, commit_id?: string }} payload
+ */
+export async function githubPostPullReview(owner, repo, pullNumber, payload) {
+  const o = encodeURIComponent(owner);
+  const r = encodeURIComponent(repo);
+  const n = encodeURIComponent(String(pullNumber));
+  return githubRestJson("POST", `/repos/${o}/${r}/pulls/${n}/reviews`, payload);
+}

package/src/github-rest.js

@@ -26,16 +26,30 @@ export function useDirectGithub() {
  */
 export async function githubRestJson(method, path, jsonBody, tokenOverride) {
   const token = tokenOverride ?? getGithubPat();
-  if (!token) {
+  const methodUpper = String(method).toUpperCase();
+  const unauthDisabled =
+    process.env.NUGIT_GITHUB_UNAUTHENTICATED === "0" ||
+    process.env.NUGIT_GITHUB_UNAUTHENTICATED === "false";
+  const allowUnauthenticatedGet =
+    !unauthDisabled &&
+    !token &&
+    (methodUpper === "GET" || methodUpper === "HEAD");
+
+  if (!token && !allowUnauthenticatedGet) {
     throw new Error(
-      "Set NUGIT_USER_TOKEN or STACKPR_USER_TOKEN (GitHub PAT or OAuth token with repo scope)"
+      "GitHub authentication required for this request. Run `nugit auth login` or `nugit auth pat --token …`, " +
+        "or set NUGIT_USER_TOKEN / STACKPR_USER_TOKEN. " +
+        "Some read-only public GETs work without a token (low rate limits); set NUGIT_GITHUB_UNAUTHENTICATED=0 to disable that."
     );
   }
   const headers = {
-    Authorization: `Bearer ${token}`,
     Accept: "application/vnd.github+json",
-    "X-GitHub-Api-Version": "2022-11-28"
+    "X-GitHub-Api-Version": "2022-11-28",
+    "User-Agent": "nugit-cli"
   };
+  if (token) {
+    headers.Authorization = `Bearer ${token}`;
+  }
   if (jsonBody !== undefined) {
     headers["Content-Type"] = "application/json";
   }
@@ -147,19 +161,125 @@ export async function githubCreatePullRequest(owner, repo, fields) {
 /**
  * @param {number} page
  */
-export async function githubListUserRepos(page = 1) {
+/**
+ * @param {number} [page]
+ * @param {number} [perPage] max 100
+ */
+export async function githubListUserRepos(page = 1, perPage = 100) {
+  const pp = Math.min(100, Math.max(1, perPage));
+  const p = Math.max(1, page);
   return githubRestJson(
     "GET",
-    `/user/repos?page=${encodeURIComponent(String(page))}&per_page=30`
+    `/user/repos?page=${encodeURIComponent(String(p))}&per_page=${encodeURIComponent(String(pp))}&sort=full_name`
   );
 }
 
+/**
+ * @returns {Promise<unknown[]>}
+ */
+export async function githubListAllUserRepos() {
+  /** @type {unknown[]} */
+  const all = [];
+  let page = 1;
+  for (;;) {
+    const chunk = await githubListUserRepos(page, 100);
+    if (!Array.isArray(chunk) || chunk.length === 0) {
+      break;
+    }
+    all.push(...chunk);
+    if (chunk.length < 100) {
+      break;
+    }
+    page += 1;
+  }
+  return all;
+}
+
+/**
+ * @param {string} owner
+ * @param {string} repo
+ * @param {string} base
+ * @param {string} head
+ */
+export async function githubCompareRefs(owner, repo, base, head) {
+  const o = encodeURIComponent(owner);
+  const r = encodeURIComponent(repo);
+  const bh = `${base}...${head}`;
+  return githubRestJson("GET", `/repos/${o}/${r}/compare/${encodeURIComponent(bh)}`);
+}
+
+/**
+ * @param {string} owner
+ * @param {string} repo
+ * @param {string} treeSha commit or tree sha
+ * @param {boolean} [recursive]
+ */
+export async function githubGetGitTree(owner, repo, treeSha, recursive = false) {
+  const o = encodeURIComponent(owner);
+  const r = encodeURIComponent(repo);
+  const sha = encodeURIComponent(treeSha);
+  const q = recursive ? "?recursive=1" : "";
+  return githubRestJson("GET", `/repos/${o}/${r}/git/trees/${sha}${q}`);
+}
+
 /**
  * Search issues (includes PRs). `q` is GitHub search query syntax.
  * @param {string} q
  * @param {number} [perPage] max 100
  * @param {number} [page] 1-based
  */
+/**
+ * GitHub repository search (`q` uses GitHub search syntax, e.g. `todo language:python` or `user:orgname`).
+ * @param {string} q
+ * @param {number} [perPage]
+ * @param {number} [page]
+ */
+export async function githubSearchRepositories(q, perPage = 20, page = 1) {
+  const pp = Math.min(100, Math.max(1, perPage));
+  const p = Math.max(1, page);
+  const query = new URLSearchParams({
+    q,
+    per_page: String(pp),
+    page: String(p)
+  });
+  return githubRestJson("GET", `/search/repositories?${query.toString()}`);
+}
+
+/**
+ * List branches in a repository (one page).
+ * @param {string} owner
+ * @param {string} repo
+ * @param {number} [page] 1-based
+ * @param {number} [perPage] max 100
+ */
+export async function githubListBranches(owner, repo, page = 1, perPage = 100) {
+  const pp = Math.min(100, Math.max(1, perPage));
+  const p = Math.max(1, page);
+  return githubRestJson(
+    "GET",
+    `/repos/${encodeURIComponent(owner)}/${encodeURIComponent(repo)}/branches?page=${p}&per_page=${pp}`
+  );
+}
+
+/**
+ * @param {string} owner
+ * @param {string} repo
+ * @returns {Promise<unknown[]>}
+ */
+export async function githubListAllBranches(owner, repo) {
+  /** @type {unknown[]} */
+  const all = [];
+  let page = 1;
+  for (;;) {
+    const chunk = await githubListBranches(owner, repo, page, 100);
+    if (!Array.isArray(chunk) || chunk.length === 0) break;
+    all.push(...chunk);
+    if (chunk.length < 100) break;
+    page += 1;
+  }
+  return all;
+}
+
 export async function githubSearchIssues(q, perPage = 30, page = 1) {
   const pp = Math.min(100, Math.max(1, perPage));
   const p = Math.max(1, page);
@@ -189,6 +309,29 @@ export async function githubListOpenPulls(owner, repo, page = 1, perPage = 30) {
   );
 }
 
+/**
+ * @param {string} owner
+ * @param {string} repo
+ * @returns {Promise<unknown[]>}
+ */
+export async function githubListAllOpenPulls(owner, repo) {
+  /** @type {unknown[]} */
+  const all = [];
+  let page = 1;
+  for (;;) {
+    const chunk = await githubListOpenPulls(owner, repo, page, 100);
+    if (!Array.isArray(chunk) || chunk.length === 0) {
+      break;
+    }
+    all.push(...chunk);
+    if (chunk.length < 100) {
+      break;
+    }
+    page += 1;
+  }
+  return all;
+}
+
 /**
  * @param {string} owner
  * @param {string} repo

package/src/nugit-config.js

@@ -0,0 +1,84 @@
+/**
+ * Config-related actions for the `nugit config` and `nugit env` commands.
+ * Extracted from the former nugit-start.js.
+ */
+import fs from "fs";
+import path from "path";
+import {
+  expandUserPath,
+  getConfigPath,
+  inferMonorepoRootFromCli,
+  readUserConfig,
+  writeUserConfig,
+  buildStartEnv
+} from "./user-config.js";
+
+/**
+ * @param {object} opts
+ * @param {string} [opts.installRoot]
+ * @param {string} [opts.envFile]
+ * @param {string} [opts.workingDirectory]
+ */
+export function runConfigInit(opts) {
+  const root = path.resolve(expandUserPath(opts.installRoot || inferMonorepoRootFromCli()));
+  const defaultEnv = path.join(root, ".env");
+  const envFile = expandUserPath(opts.envFile || defaultEnv);
+  const cfg = { installRoot: root, envFile };
+  if (opts.workingDirectory) cfg.workingDirectory = expandUserPath(opts.workingDirectory);
+  writeUserConfig(cfg);
+  console.error(`Wrote ${getConfigPath()}`);
+  console.error(`  installRoot: ${root}`);
+  console.error(`  envFile: ${envFile}`);
+  if (!fs.existsSync(envFile)) {
+    console.error("  (env file does not exist yet — create it or run: nugit config set env-file <path>)");
+  }
+  if (cfg.workingDirectory) console.error(`  workingDirectory: ${cfg.workingDirectory}`);
+}
+
+export function runConfigShow() {
+  console.log(JSON.stringify(readUserConfig(), null, 2));
+}
+
+/**
+ * @param {string} key
+ * @param {string} value
+ */
+export function runConfigSet(key, value) {
+  const c = readUserConfig();
+  const k = key.toLowerCase().replace(/_/g, "-");
+  if (k === "install-root") {
+    c.installRoot = path.resolve(expandUserPath(value));
+  } else if (k === "env-file") {
+    c.envFile = expandUserPath(value);
+  } else if (k === "working-directory" || k === "cwd") {
+    c.workingDirectory = expandUserPath(value);
+  } else {
+    throw new Error(`Unknown key "${key}". Use: install-root | env-file | working-directory`);
+  }
+  writeUserConfig(c);
+  console.error(`Updated ${getConfigPath()}`);
+}
+
+/**
+ * @param {"bash" | "fish"} shell
+ */
+export function runEnvExport(shell) {
+  const cfg = readUserConfig();
+  let env;
+  try {
+    env = buildStartEnv(cfg);
+  } catch (e) {
+    console.error(String(e?.message || e));
+    process.exit(1);
+  }
+  const entries = Object.entries(env);
+  if (shell === "fish") {
+    for (const [k, v] of entries) {
+      if (v !== undefined) console.log(`set -gx ${k} ${JSON.stringify(v)};`);
+    }
+  } else {
+    for (const [k, v] of entries) {
+      if (v !== undefined) console.log(`export ${k}=${JSON.stringify(v)}`);
+    }
+  }
+}