Skip to content

Bump omercnet/pr-radar to 1.2.1 - #162

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
bump/omercnet/pr-radar-1.2.1
Open

github-actions[bot] wants to merge 1 commit into
mainfrom
bump/omercnet/pr-radar-1.2.1

Conversation

@github-actions

@github-actions github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

omercnet/pr-radar: {"kind":"npm","package":"@omercnet/paseo-pr-radar","version":"1.2.0","resolved":"https://registry.npmjs.org/@omercnet/paseo-pr-radar/-/paseo-pr-radar-1.2.0.tgz","integrity":"sha512-sk+zuzTGZ5Vmri2toic3z3uChm4d6kEShSr/UjkqfOTeILrsZVDHUc+XKX/CtZIQazT/vWvbgZ115+KUiAonCw=="} -> {"kind":"npm","package":"@omercnet/paseo-pr-radar","version":"1.2.1","resolved":"https://registry.npmjs.org/@omercnet/paseo-pr-radar/-/paseo-pr-radar-1.2.1.tgz","integrity":"sha512-wYK8r2UoAy90ZrMn9EfEKqsSG6yil8UFwdOi4igJBVLGtA+rUKCLr9wv3/L5NhT3SiJ0XyoPLvsGWIyfBGwxVQ=="}
Provenance verified: built from https://github.com/omercnet/paseo-plugins/tree/HEAD/pr-radar at 8e8dfa1aebbe7c23673f344e75d93dfac91daba9.
Submitted by @omercnet.

Merging approves this version. The published index keeps pointing at the previous one until then.

Artifact diff
diff -ruN a/package/client/pr-radar.tsx b/package/client/pr-radar.tsx
--- a/package/client/pr-radar.tsx	1985-10-26 08:15:00.000000000 +0000
+++ b/package/client/pr-radar.tsx	1985-10-26 08:15:00.000000000 +0000
@@ -590,7 +590,7 @@
         : savedView
           ? `No pull requests match the ${SAVED_VIEW_TITLES[savedView].toLowerCase()} view.`
           : selected
-            ? `No pull requests are ${BUCKET_TITLES[selected].toLowerCase()}.`
+            ? "No pull requests are in this queue."
             : "No open pull requests are visible to GitHub or linked to a Paseo workspace.";
   const summaryMetrics = [
     { label: "Action now", value: counts["needs-you"] },
diff -ruN a/package/package.json b/package/package.json
--- a/package/package.json	1985-10-26 08:15:00.000000000 +0000
+++ b/package/package.json	1985-10-26 08:15:00.000000000 +0000
@@ -1,6 +1,6 @@
 {
   "name": "@omercnet/paseo-pr-radar",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "type": "module",
   "description": "A viewer-aware delivery queue for pull requests linked to Paseo workspaces.",
   "license": "MIT",
@@ -46,10 +46,10 @@
   },
   "devDependencies": {
     "@biomejs/biome": "^2.5.10",
-    "@getpaseo/cli": "0.11.0-beta.3",
-    "@getpaseo/client": "0.11.0-beta.3",
-    "@getpaseo/plugin": "0.11.0-beta.3",
-    "@getpaseo/protocol": "0.11.0-beta.3",
+    "@getpaseo/cli": "0.11.0-beta.5",
+    "@getpaseo/client": "0.11.0-beta.5",
+    "@getpaseo/plugin": "0.11.0-beta.5",
+    "@getpaseo/protocol": "0.11.0-beta.5",
     "@tanstack/react-query": "^5.102.3",
     "@types/node": "^24.10.1",
     "@types/react": "^19.2.18",
diff -ruN a/package/server/viewer-scope.ts b/package/server/viewer-scope.ts
--- a/package/server/viewer-scope.ts	1985-10-26 08:15:00.000000000 +0000
+++ b/package/server/viewer-scope.ts	1985-10-26 08:15:00.000000000 +0000
@@ -11,9 +11,13 @@
 const ENRICHMENT_BATCH_SIZE = 20;
 const COMMAND_TIMEOUT_MS = 15_000;
 const MAX_BUFFER_BYTES = 16 * 1024 * 1024;
-const statePath = join(homedir(), ".paseo", "plugin-data", "pr-radar", "inbox-state.json");
 const SCOPE_SEARCH_LIMIT = 1000;
 
+function statePath(): string {
+  const home = process.env.PASEO_HOME ?? join(homedir(), ".paseo");
+  return join(home, "plugin-data", "pr-radar", "inbox-state.json");
+}
+
 interface SearchRecord {
   id: string;
   number: number;
@@ -254,7 +258,7 @@
 
 async function readState(): Promise<StoredState> {
   try {
-    const state = JSON.parse(await readFile(statePath, "utf8")) as StoredState;
+    const state = JSON.parse(await readFile(statePath(), "utf8")) as StoredState;
     if (state.version === 2 && state.windows && typeof state.windows === "object") return state;
   } catch (error) {
     if ((error as NodeJS.ErrnoException).code !== "ENOENT") {
@@ -265,10 +269,11 @@
 }
 
 async function writeState(state: StoredState): Promise<void> {
-  await mkdir(dirname(statePath), { recursive: true });
-  const temporaryPath = `${statePath}.${process.pid}.tmp`;
+  const path = statePath();
+  await mkdir(dirname(path), { recursive: true });
+  const temporaryPath = `${path}.${process.pid}.tmp`;
   await writeFile(temporaryPath, `${JSON.stringify(state)}\n`, { mode: 0o600 });
-  await rename(temporaryPath, statePath);
+  await rename(temporaryPath, path);
 }
 
 function storedItem(item: GitHubInboxItem): StoredItem {

This version has no OVERVIEW.md. The registry requires one to update a listing; the bump cannot merge until the repository adds it.

Inline validation failed. See the Bump workflow log.

@github-actions github-actions Bot added the bump Pinned artifact update label Oct 7, 2026

@paseo-bot paseo-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs from you: nothing, changes requested

@omercnet, @omercnet/paseo-pr-radar 1.2.1 can't merge yet: the package has no OVERVIEW.md, and the registry needs one for every new version.

Change needed: add OVERVIEW.md next to paseo-plugin.json in pr-radar/, include it in the published package, and release a new version; the bump workflow opens a fresh pull request for it. Your file replaces the registry copy plugins/omercnet/pr-radar.md, which you can start from without its last "imported from paseo.cafe" line. The shape is in REVIEW.md: what it is and does, how it works if needed, setup, then what it reads or sends and known limits, with no installation steps, badges, or changelog.

The code change in 1.2.1 is fine. Nothing else needs to change.

Review data

  • Artifact: npm @omercnet/paseo-pr-radar 1.2.1 (from 1.2.0), https://registry.npmjs.org/@omercnet/paseo-pr-radar/-/paseo-pr-radar-1.2.1.tgz. npm provenance names omercnet/paseo-plugins at 8e8dfa1aebbe7c23673f344e75d93dfac91daba9, built by .github/workflows/release-please.yml on main.
  • Integrity: SHA-512 of both tarballs matches the record (old and new pin), npm's dist.integrity, and the provenance subject digest.
  • Validation: npm test passes (140/140). node scripts/validate.ts --online --changed fails with one error: omercnet/pr-radar/OVERVIEW.md is required.
  • Extracted and inspected: both tarballs (14 files each), extracted and diffed locally. client/pr-radar.tsx:593 changes the empty-queue text. server/viewer-scope.ts:16-19 moves the plugin's state file to plugin-data/pr-radar/inbox-state.json under PASEO_HOME (default: the Paseo home directory), read at server/viewer-scope.ts:261 and written with mode 0600 at server/viewer-scope.ts:272-276. package.json changes the version and moves dev @getpaseo/* to 0.11.0-beta.5.
  • Install-time commands: none. paseo-plugin.json has no install or build; package.json has no preinstall, install, postinstall, or prepare.
  • Dependencies: no runtime dependencies. The monorepo has bun.lock and no npm lockfile: all 1076 registry entries carry sha512 integrity, with no git or tarball sources. This package has no install or build command, so no repository lockfile is used at install; Paseo installs the published package with --ignore-scripts.
  • Hosts: none added; GitHub is reached only through the gh CLI.
  • Credentials and environment: one new read, PASEO_HOME (server/viewer-scope.ts:17), to locate the plugin's own state file. The existing environment passed to gh with GIT_TERMINAL_PROMPT=0 (server/viewer-scope.ts:68) is unchanged.
  • Filesystem: the state file stays in the plugin's own storage under the Paseo home. No other change.
  • Execution: the existing execFile("gh", ...) (server/viewer-scope.ts:65) is unchanged. No eval, new Function, or dynamic import().
  • Runtime installs: none.
  • Source match: every file is byte-identical to pr-radar/ at 8e8dfa1 except package.json, where catalog: specifiers are replaced by the root catalog's versions. Readable TypeScript, nothing bundled.
  • Listing media: pr-radar-github-inbox-wide.png and pr-radar-github-inbox-compact.png, each HTTP 200 image/png.
  • Overview: missing in the 1.2.1 package and in pr-radar/ at 8e8dfa1. The registry copy still describes 1.2.1.
  • Decision: changes requested, because OVERVIEW.md is missing.
  • Reviewed commit: e19b967c876c3d23db33426967fd6a87ff015f8b

@paseo-bot paseo-bot Bot mentioned this pull request Oct 7, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump Pinned artifact update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants