Skip to content

Bump omercnet/agent-monitor to 1.2.1 - #159

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

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

Conversation

@github-actions

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

Copy link
Copy Markdown
Contributor

omercnet/agent-monitor: {"kind":"npm","package":"@omercnet/paseo-agent-monitor","version":"1.2.0","resolved":"https://registry.npmjs.org/@omercnet/paseo-agent-monitor/-/paseo-agent-monitor-1.2.0.tgz","integrity":"sha512-dY7/3f49sPMVwb+Wr7iUsw4jCoQWRX2UF9Li+uU8yJdN0cj4BvTsz4GDiR3OsKk8zJ1/o01DSRi1e4uDRS3TmA=="} -> {"kind":"npm","package":"@omercnet/paseo-agent-monitor","version":"1.2.1","resolved":"https://registry.npmjs.org/@omercnet/paseo-agent-monitor/-/paseo-agent-monitor-1.2.1.tgz","integrity":"sha512-/UMJU4y8uymVZ1S5AhslWroqwE5qELcL0mxAfnqrNWyEAjqvIQW97CcCM7Pt5hGk8KUdMgDUXpdjGa0XtX7iww=="}
Provenance verified: built from https://github.com/omercnet/paseo-plugins/tree/HEAD/agent-monitor 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/agent-monitor.tsx b/package/client/agent-monitor.tsx
--- a/package/client/agent-monitor.tsx	1985-10-26 08:15:00.000000000 +0000
+++ b/package/client/agent-monitor.tsx	1985-10-26 08:15:00.000000000 +0000
@@ -16,8 +16,8 @@
   childCounts,
   type MonitorDirectory,
   matches,
-  PARENT_AGENT_ID_LABEL,
   type ProjectGroup,
+  parentAgentId,
   parseBucketParam,
   placement,
   shouldCollapseWorkspace,
@@ -509,7 +509,7 @@
           {group.entries.map((entry) => (
             <View
               key={entry.agent.id}
-              style={PARENT_AGENT_ID_LABEL in entry.agent.labels ? styles.childRow : undefined}
+              style={parentAgentId(entry.agent) !== null ? styles.childRow : undefined}
             >
               {renderRow({ item: entry })}
             </View>
diff -ruN a/package/client/monitor.ts b/package/client/monitor.ts
--- a/package/client/monitor.ts	1985-10-26 08:15:00.000000000 +0000
+++ b/package/client/monitor.ts	1985-10-26 08:15:00.000000000 +0000
@@ -64,6 +64,14 @@
 
 export const PARENT_AGENT_ID_LABEL = "paseo.parent-agent-id";
 
+/** Prefers the first-class snapshot field; older hosts only carry the legacy label. */
+export function parentAgentId(agent: AgentSnapshot): string | null {
+  const firstClass = "parentAgentId" in agent ? agent.parentAgentId : null;
+  if (typeof firstClass === "string" && firstClass.trim()) return firstClass.trim();
+  const legacy = agent.labels?.[PARENT_AGENT_ID_LABEL];
+  return typeof legacy === "string" && legacy.trim() ? legacy.trim() : null;
+}
+
 export type Bucket = "attention" | "running" | "idle" | "closed";
 
 export const BUCKETS: readonly Bucket[] = ["attention", "running", "idle", "closed"];
@@ -199,8 +207,8 @@
 
 export function sortEntries(entries: AgentEntry[], agentSort: AgentSort): void {
   entries.sort((left, right) => {
-    const leftChild = PARENT_AGENT_ID_LABEL in left.agent.labels;
-    const rightChild = PARENT_AGENT_ID_LABEL in right.agent.labels;
+    const leftChild = parentAgentId(left.agent) !== null;
+    const rightChild = parentAgentId(right.agent) !== null;
     if (leftChild !== rightChild) return leftChild ? 1 : -1;
     return compareBySort(left, right, agentSort);
   });
@@ -396,7 +404,7 @@
 export function childCounts(entries: readonly AgentEntry[]): ReadonlyMap<string, number> {
   const counts = new Map<string, number>();
   for (const entry of entries) {
-    const parentId = entry.agent.labels[PARENT_AGENT_ID_LABEL];
+    const parentId = parentAgentId(entry.agent);
     if (parentId) counts.set(parentId, (counts.get(parentId) ?? 0) + 1);
   }
   return counts;
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-agent-monitor",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "type": "module",
   "description": "A Paseo plugin that provides a host-wide agent triage roster.",
   "license": "MIT",
@@ -45,10 +45,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",

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-agent-monitor 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 agent-monitor/, 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/agent-monitor.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-agent-monitor 1.2.1 (from 1.2.0), https://registry.npmjs.org/@omercnet/paseo-agent-monitor/-/paseo-agent-monitor-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/agent-monitor/OVERVIEW.md is required.
  • Extracted and inspected: both tarballs (16 files each), extracted and diffed locally. client/monitor.ts:67-73 adds parentAgentId(agent), which prefers the snapshot's parentAgentId field over the legacy paseo.parent-agent-id label, used at client/monitor.ts:210-211, client/monitor.ts:407, and client/agent-monitor.tsx:512. 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; no URL literals or network calls.
  • Credentials and environment: none.
  • Filesystem: none.
  • Execution: no child_process, eval, new Function, or dynamic import().
  • Runtime installs: none.
  • Source match: every file is byte-identical to agent-monitor/ at 8e8dfa1 except package.json, where catalog: specifiers are replaced by the root catalog's versions. Readable TypeScript, nothing bundled.
  • Listing media: agent-monitor-roster.png and agent-monitor-settings.png, each HTTP 200 image/png.
  • Overview: missing in the 1.2.1 package and in agent-monitor/ at 8e8dfa1. The registry copy still describes 1.2.1.
  • Decision: changes requested, because OVERVIEW.md is missing.
  • Reviewed commit: 464cead7ca1ad049c78efd01e6245e8c0b8bdc63

@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