Repository navigation
Bump omercnet/pr-radar to 1.2.1 - #162
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-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-radar1.2.1 (from 1.2.0),https://registry.npmjs.org/@omercnet/paseo-pr-radar/-/paseo-pr-radar-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/pr-radar/OVERVIEW.md is required. - Extracted and inspected: both tarballs (14 files each), extracted and diffed locally.
client/pr-radar.tsx:593changes the empty-queue text.server/viewer-scope.ts:16-19moves the plugin's state file toplugin-data/pr-radar/inbox-state.jsonunderPASEO_HOME(default: the Paseo home directory), read atserver/viewer-scope.ts:261and written with mode 0600 atserver/viewer-scope.ts:272-276.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 added; GitHub is reached only through the
ghCLI. - 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 toghwithGIT_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. Noeval,new Function, or dynamicimport(). - Runtime installs: none.
- Source match: every file is byte-identical to
pr-radar/at8e8dfa1exceptpackage.json, wherecatalog:specifiers are replaced by the root catalog's versions. Readable TypeScript, nothing bundled. - Listing media:
pr-radar-github-inbox-wide.pngandpr-radar-github-inbox-compact.png, each HTTP 200image/png. - Overview: missing in the 1.2.1 package and in
pr-radar/at8e8dfa1. The registry copy still describes 1.2.1. - Decision: changes requested, because
OVERVIEW.mdis missing. - Reviewed commit:
e19b967c876c3d23db33426967fd6a87ff015f8b
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
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.