@blogic-cz/agent-tools 0.15.11 → 0.15.13

package/README.md

@@ -263,7 +263,7 @@ Pod `exec` is limited to direct `redis-cli PING/INFO` and `ls` diagnostics. Gene
 
 ### gh-tool machine contracts
 
-`pr view` adds `headSha` and `baseSha`; failed-check evidence adds the same SHA pair. Review summaries, inline comments, and threads add `commitSha` plus `feedbackOrigin`: `current_head` only for an exact `commitSha === headSha`, `pre_existing` for a different known SHA (not an obsolescence verdict), and `unknown` when either SHA is absent. Issue comments always use `commitSha: null` and `feedbackOrigin: unknown`. `review-triage` preserves existing fields and adds `inlineComments` plus per-kind `feedbackOriginCounts`; batch triage returns the same object per PR. `pr request-review --reviewers alice,bob` emits sorted `submittedReviewers` (normalized input) and `requestedReviewers` (GitHub-confirmed result).
+`pr view` adds `headSha` and `baseSha`; failed-check evidence adds the same SHA pair. Review summaries, inline comments, and threads add `commitSha` plus `feedbackOrigin`: `current_head` only for an exact `commitSha === headSha`, `pre_existing` for a different known SHA (not an obsolescence verdict), and `unknown` when either SHA is absent. Issue comments always use `commitSha: null` and `feedbackOrigin: unknown`. `review-triage` preserves existing fields and adds `inlineComments` plus per-kind `feedbackOriginCounts`; batch triage returns the same object per PR. `pr request-review --reviewers alice,bob` emits sorted `submittedReviewers` (normalized input), `newlyRequested` and `alreadyPending` (the submitted logins split by whether a pending request already existed, so a fresh re-request is distinguishable from a no-op), and `requestedReviewers` (GitHub-confirmed result). `pr last-human-reviewer` derives `currentRequestedReviewers` from live `reviewRequests` only; timeline events are not replayed because GitHub clears a pending request on review submit without emitting `ReviewRequestRemovedEvent`.
 
 `pr watch --prs 12,34 --until terminal --format jsonl` accepts at most 50 unique, digits-only PR numbers and emits only JSONL state transitions. Identity uses `repo/pr/headSha/runId/attempt/jobId`; `runId`, `attempt`, and `jobId` are omitted from an event rather than emitted as `null`, and `checkId` is no longer emitted. State/bucket revisions emit even when identity stays stable; `supersedes` appears only when identity changes. Open PRs with no checks become terminal only after three stable empty snapshots, allowing bounded GitHub eventual consistency, and carry `checksObserved: false` — a terminal snapshot with no observed check is never green evidence. `pr checks`, batch checks, triage, and batch triage keep stderr silent with `--format json`; JSONL watch is also informationally silent. Failures still return structured nonzero errors on stderr.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blogic-cz/agent-tools",
-  "version": "0.15.11",
+  "version": "0.15.13",
   "description": "CLI tools for AI coding agent workflows — GitHub, database, Kubernetes, Azure DevOps, logs, sessions, and audit",
   "keywords": [
     "agent",

package/src/db-tool/psql.ts

@@ -0,0 +1,27 @@
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+
+// Homebrew keeps libpq keg-only, so psql is routinely installed yet absent from PATH.
+const KEG_ONLY_PSQL_DIRECTORIES = ["/opt/homebrew/opt/libpq/bin", "/usr/local/opt/libpq/bin"];
+
+export const PSQL_MISSING_HINT =
+  "psql was not found on PATH. On macOS install it with `brew install libpq`; libpq is keg-only, so also add /opt/homebrew/opt/libpq/bin (Apple silicon) or /usr/local/opt/libpq/bin (Intel) to PATH.";
+
+export const resolvePsqlSearchPath = (
+  pathEnv: string | undefined,
+  fileExists: (candidate: string) => boolean = existsSync,
+): string | undefined => {
+  const directories = (pathEnv ?? "").split(":").filter((directory) => directory.length > 0);
+  if (directories.some((directory) => fileExists(join(directory, "psql")))) {
+    return pathEnv;
+  }
+
+  const kegOnly = KEG_ONLY_PSQL_DIRECTORIES.find((directory) =>
+    fileExists(join(directory, "psql")),
+  );
+  if (kegOnly === undefined) {
+    return pathEnv;
+  }
+
+  return directories.length > 0 ? [...directories, kegOnly].join(":") : kegOnly;
+};

package/src/db-tool/service.ts

@@ -10,6 +10,7 @@ import { resolveEnvironmentScopedPrerequisites } from "#shared/prerequisites/con
 import { runWithProfilePrerequisites } from "#shared/prerequisites/runtime";
 import { buildApiProbeArgs } from "#shared/k8s-probe";
 import { DbConfigService, TUNNEL_CHECK_INTERVAL_MS } from "./config-service";
+import { PSQL_MISSING_HINT, resolvePsqlSearchPath } from "./psql";
 import {
   DbConnectionError,
   DbMutationBlockedError,
@@ -235,7 +236,8 @@ export class DbService extends Context.Service<
                 new DbQueryError({
                   message: `Command execution failed: ${String(platformError)}`,
                   sql: "shell command",
-                  stderr: undefined,
+                  stderr: String(platformError),
+                  ...(String(platformError).includes("psql") ? { hint: PSQL_MISSING_HINT } : {}),
                 }),
             ),
           );
@@ -402,6 +404,7 @@ export class DbService extends Context.Service<
             stderr: "pipe",
             env: {
               ...process.env,
+              PATH: resolvePsqlSearchPath(process.env.PATH),
               ...(password ? { PGPASSWORD: password } : {}),
               ...(isFullyReadOnly(config)
                 ? { PGOPTIONS: "-c default_transaction_read_only=on" }

package/src/gh-tool/pr/commands.ts

@@ -141,6 +141,15 @@ export const requestReview = (pr: number, reviewers: string) =>
     const submittedReviewers = yield* parseReviewers(reviewers);
     const gh = yield* GitHubService;
     const repo = yield* gh.getRepoInfo();
+    // ponytail: not atomic — a concurrent request between GET and POST misreports newlyRequested.
+    const pendingBefore = new Set(
+      (yield* confirmedReviewers(
+        yield* gh.runGhJson<RequestedReviewersResponse>([
+          "api",
+          `repos/${repo.owner}/${repo.name}/pulls/${pr}`,
+        ]),
+      )).map((login) => login.toLowerCase()),
+    );
     const response = yield* gh.runGhJson<RequestedReviewersResponse>([
       "api",
       "--method",
@@ -151,6 +160,8 @@ export const requestReview = (pr: number, reviewers: string) =>
     return {
       pr,
       submittedReviewers,
+      newlyRequested: submittedReviewers.filter((reviewer) => !pendingBefore.has(reviewer)),
+      alreadyPending: submittedReviewers.filter((reviewer) => pendingBefore.has(reviewer)),
       requestedReviewers: yield* confirmedReviewers(response),
     };
   });

package/src/gh-tool/pr/review.ts

@@ -126,27 +126,6 @@ const LAST_HUMAN_REVIEWER_QUERY = `
             submittedAt
           }
         }
-        timelineItems(first: 250, itemTypes: [REVIEW_REQUESTED_EVENT, REVIEW_REQUEST_REMOVED_EVENT]) {
-          nodes {
-            __typename
-            ... on ReviewRequestedEvent {
-              requestedReviewer {
-                __typename
-                ... on User { login }
-                ... on Team { name }
-                ... on Bot { login }
-              }
-            }
-            ... on ReviewRequestRemovedEvent {
-              requestedReviewer {
-                __typename
-                ... on User { login }
-                ... on Team { name }
-                ... on Bot { login }
-              }
-            }
-          }
-        }
       }
     }
   }
@@ -237,12 +216,6 @@ type LastHumanReviewerQueryResult = {
           submittedAt: string | null;
         }>;
       };
-      timelineItems: {
-        nodes: Array<{
-          __typename: string;
-          requestedReviewer: RequestedReviewerNode;
-        }>;
-      };
     };
   };
 };
@@ -1066,30 +1039,11 @@ export const fetchLastHumanReviewer = Effect.fn("pr.fetchLastHumanReviewer")(fun
 
   const prNode = response.repository.pullRequest;
 
-  const requestedFromTimeline: string[] = [];
-  for (const item of prNode.timelineItems.nodes) {
-    const login = requestedReviewerLogin(item.requestedReviewer);
-    if (login === null) {
-      continue;
-    }
-    if (item.__typename === "ReviewRequestedEvent") {
-      if (!requestedFromTimeline.includes(login)) {
-        requestedFromTimeline.push(login);
-      }
-    } else {
-      const index = requestedFromTimeline.indexOf(login);
-      if (index !== -1) {
-        requestedFromTimeline.splice(index, 1);
-      }
-    }
-  }
-
-  const currentRequestedReviewers =
-    requestedFromTimeline.length > 0
-      ? requestedFromTimeline
-      : prNode.reviewRequests.nodes
-          .map((node) => requestedReviewerLogin(node.requestedReviewer))
-          .filter((login): login is string => login !== null);
+  // GitHub clears a pending request on review submit without a ReviewRequestRemovedEvent, so
+  // only reviewRequests is authoritative; timeline replay would report stale reviewers forever.
+  const currentRequestedReviewers = prNode.reviewRequests.nodes
+    .map((node) => requestedReviewerLogin(node.requestedReviewer))
+    .filter((login): login is string => login !== null);
 
   const latestHumanReview = prNode.reviews.nodes.reduce<{ login: string; at: string } | null>(
     (latest, review) => {

package/src/k8s-tool/service.ts

@@ -141,7 +141,7 @@ export class K8sService extends Context.Service<
                   message: `Command execution failed: ${String(platformError)}`,
                   command: commandStr,
                   exitCode: -1,
-                  stderr: undefined,
+                  stderr: String(platformError),
                 }),
             ),
           );
@@ -320,7 +320,7 @@ export class K8sService extends Context.Service<
                       message: `Command execution failed: ${String(platformError)}`,
                       command: fullCommand,
                       exitCode: -1,
-                      stderr: undefined,
+                      stderr: String(platformError),
                     }),
                 ),
               );