chore: refresh SDK dependencies and block structure - #42
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates several SDK dependencies, upgrading @platforma-sdk/block-tools to 2.12.7 and @platforma-sdk/test to 1.80.5, along with an update to @milaboratories/pl-middle-layer to 1.66.4. It also downgrades the packageManager version in package.json to pnpm@9.12.0 and adds a corresponding changeset file. No review comments were provided, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| "oxlint": "*" | ||
| }, | ||
| "packageManager": "pnpm@9.14.4+sha512.c8180b3fbe4e4bca02c94234717896b5529740a6cbadf19fa78254270403ea2f27d4e1d46a08a0f56c89b63dc8ebfd3ee53326da720273794e6200fcf0d184ab" | ||
| "packageManager": "pnpm@9.12.0" |
There was a problem hiding this comment.
pnpm version downgraded and integrity hash removed
The packageManager field changed from pnpm@9.14.4+sha512.c8180b3... to pnpm@9.12.0. This is a two-part regression: (1) the version is rolled back by two minor releases, which will cause any developer or CI runner relying on corepack to switch to an older binary; (2) the sha512 hash is dropped entirely, removing the cryptographic guarantee that corepack downloads the exact trusted binary. Without the hash, corepack will accept any pnpm binary that self-reports as 9.12.0. Was this change intentional, or did block-tools structure refresh inadvertently overwrite the field?
Prompt To Fix With AI
This is a comment left during a code review.
Path: package.json
Line: 35
Comment:
**pnpm version downgraded and integrity hash removed**
The `packageManager` field changed from `pnpm@9.14.4+sha512.c8180b3...` to `pnpm@9.12.0`. This is a two-part regression: (1) the version is rolled back by two minor releases, which will cause any developer or CI runner relying on corepack to switch to an older binary; (2) the sha512 hash is dropped entirely, removing the cryptographic guarantee that corepack downloads the exact trusted binary. Without the hash, corepack will accept any pnpm binary that self-reports as 9.12.0. Was this change intentional, or did `block-tools structure refresh` inadvertently overwrite the field?
How can I resolve this? If you propose a fix, please make it concise.
Refresh SDK dependencies and block structure via
pnpm upgrade-sdk.This runs:
block-tools structure refresh --update-deps-only+pnpm iblock-tools structure refresh+pnpm ipnpm fmtIncludes a patch changeset ("SDK Update").
Greptile Summary
This PR upgrades SDK dependencies via
pnpm upgrade-sdk, bumping@platforma-sdk/block-tools(2.12.6→2.12.7),@platforma-sdk/test(1.80.3→1.80.5), and the transitive@milaboratories/pl-middle-layer(1.66.3→1.66.4). A patch changeset marks all four block packages for release, but thepackage.jsonpackageManagerfield was unintentionally overwritten as a side-effect.@platforma-sdk/block-tools2.12.6→2.12.7 and@platforma-sdk/test1.80.3→1.80.5: straightforward patch/minor bumps reflected consistently acrosspnpm-workspace.yamlandpnpm-lock.yaml.packageManagerfield regressed: changed frompnpm@9.14.4+sha512.c8180b3...topnpm@9.12.0— the version was rolled back two minor releases and the sha512 integrity hash was stripped, weakening supply-chain verification for anyone using corepack.Touched Terms:
packageManagerpackage.jsonfield consumed by corepack to pin the exact package manager binary (version + sha512 hash) for a repopnpm@9.14.4+sha512.c8180b3…topnpm@9.12.0; integrity hash removed@platforma-sdk/block-tools2.12.6to2.12.7in catalog and lockfile@platforma-sdk/test1.80.3to1.80.5(skips 1.80.4) in catalog and lockfile@milaboratories/pl-middle-layer@platforma-sdk/test1.66.3to1.66.4in lockfile snapshotscatalogpnpm-workspace.yamlconsumed by all workspace packages viaspecifier: 'catalog:'block-tools,test) bumped to reflect new SDK versionsConfidence Score: 4/5
The SDK bumps are clean and consistent across all files, but the packageManager field was inadvertently downgraded and its integrity hash stripped.
The dependency updates are well-formed and internally consistent. The one real issue is in package.json: the block-tools refresh command appears to have overwritten the packageManager field, rolling it back from pnpm 9.14.4 (with a corepack-verifiable sha512 hash) to pnpm 9.12.0 without a hash. This means any engineer or CI pipeline using corepack will silently switch to an older pnpm binary and lose cryptographic verification of it. The SDK changes themselves look safe to ship once the packageManager regression is resolved.
package.json — the packageManager field needs to be restored to pnpm@9.14.4 with its original sha512 hash (or updated to the intended newer version with a fresh hash).
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A["pnpm upgrade-sdk\n(block-tools CLI)"] --> B["block-tools structure refresh\n--update-deps-only"] B --> C["pnpm i"] A --> D["block-tools structure refresh"] D --> E["pnpm i"] A --> F["pnpm fmt"] C --> G["pnpm-workspace.yaml\ncatalog updated"] E --> G G --> H["@platforma-sdk/block-tools\n2.12.6 → 2.12.7"] G --> I["@platforma-sdk/test\n1.80.3 → 1.80.5"] H --> J["pnpm-lock.yaml\nlockfile regenerated"] I --> J J --> K["@milaboratories/pl-middle-layer\n1.66.3 → 1.66.4\n(transitive)"] B --> L["package.json modified\n⚠️ packageManager\npnpm@9.14.4+sha512 → pnpm@9.12.0\n(downgrade + hash removed)"]%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%% flowchart TD A["pnpm upgrade-sdk\n(block-tools CLI)"] --> B["block-tools structure refresh\n--update-deps-only"] B --> C["pnpm i"] A --> D["block-tools structure refresh"] D --> E["pnpm i"] A --> F["pnpm fmt"] C --> G["pnpm-workspace.yaml\ncatalog updated"] E --> G G --> H["@platforma-sdk/block-tools\n2.12.6 → 2.12.7"] G --> I["@platforma-sdk/test\n1.80.3 → 1.80.5"] H --> J["pnpm-lock.yaml\nlockfile regenerated"] I --> J J --> K["@milaboratories/pl-middle-layer\n1.66.3 → 1.66.4\n(transitive)"] B --> L["package.json modified\n⚠️ packageManager\npnpm@9.14.4+sha512 → pnpm@9.12.0\n(downgrade + hash removed)"]Prompt To Fix All With AI
Reviews (1): Last reviewed commit: "chore: refresh SDK dependencies and bloc..." | Re-trigger Greptile
Context used: