Repository navigation
Bump omercnet/agent-monitor to 1.2.1 - #159
github-actions[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
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-monitor1.2.1 (from 1.2.0),https://registry.npmjs.org/@omercnet/paseo-agent-monitor/-/paseo-agent-monitor-1.2.1.tgz. npm provenance namesomercnet/paseo-pluginsat8e8dfa1aebbe7c23673f344e75d93dfac91daba9, built by.github/workflows/release-please.ymlonmain. - Integrity: SHA-512 of both tarballs matches the record (old and new pin), npm's
dist.integrity, and the provenance subject digest. - Validation:
npm testpasses (140/140).node scripts/validate.ts --online --changedfails 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-73addsparentAgentId(agent), which prefers the snapshot'sparentAgentIdfield over the legacypaseo.parent-agent-idlabel, used atclient/monitor.ts:210-211,client/monitor.ts:407, andclient/agent-monitor.tsx:512.package.jsonchanges the version and moves dev@getpaseo/*to 0.11.0-beta.5. - Install-time commands: none.
paseo-plugin.jsonhas noinstallorbuild;package.jsonhas nopreinstall,install,postinstall, orprepare. - Dependencies: no runtime dependencies. The monorepo has
bun.lockand no npm lockfile: all 1076 registry entries carry sha512integrity, with no git or tarball sources. This package has noinstallorbuildcommand, 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 dynamicimport(). - Runtime installs: none.
- Source match: every file is byte-identical to
agent-monitor/at8e8dfa1exceptpackage.json, wherecatalog:specifiers are replaced by the root catalog's versions. Readable TypeScript, nothing bundled. - Listing media:
agent-monitor-roster.pngandagent-monitor-settings.png, each HTTP 200image/png. - Overview: missing in the 1.2.1 package and in
agent-monitor/at8e8dfa1. The registry copy still describes 1.2.1. - Decision: changes requested, because
OVERVIEW.mdis missing. - Reviewed commit:
464cead7ca1ad049c78efd01e6245e8c0b8bdc63
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
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.