Repository navigation
chore(cli): vendor v3.6.2 and release plugin v1.2.6 - #10
Conversation
[Low impact] This distribution PR appears safe to merge under the supplied maintainer-approved scope decision.
|
Comments Outside DiffThese findings could not be posted inline.
|
|
@greptile please re-review current head The missing response validation in This PR distributes the already-published npm Proposed disposition: defer the acknowledged response-validation work to the CLI source and preserve the published bundle in this distribution PR. Please reassess whether it should block this release-distribution PR given the maintainer's scope decision, and report any remaining actionable distribution or metadata defects. Do not treat the validation work as fixed. |
Tested 1 flow, found no issues. What we tested
|
|
@greptile review this PR again in full on current head Consider the maintainer-approved proposed disposition: defer API response-validation hardening to the CLI source and a future release, while preserving the immutable npm 3.6.2 artifact in this distribution PR. That source issue is acknowledged and remains unfixed; please reassess whether it blocks distribution under that scope decision and check for any remaining distribution or metadata defects. The bundle is unchanged and byte-identical to published npm 3.6.2. Both CLI Check and MCP Check passed. This is a request for a full code re-review and updated merge-readiness assessment. |
Gravernloop summary
Review comments addressed
|
Vendor the published npm
greptile@3.6.2bundle and bump the plugin from 1.2.5 to 1.2.6 so plugin users receive the updated CLI.Validation: the vendored bundle reports 3.6.2, matches npm SHA-256
82e04c3f76fe06780b41dce5ca81163ca60dd5e53084269a4fdeee3e588c0a28, and the metadata passesgit diff --check. The published generated bundle contains whitespace flagged by Git; it is preserved verbatim to match npm. Require CLI Check and MCP Check before merging.CLI release notes.