Bump all dependencies to latest versions#80
Conversation
Upgrade all deps including major bumps: shared-ui 0.4.5→0.6.3, typeberry/lib 0.5.1→0.5.10, vite 7→8, jsdom 27→29, typescript 5.6→5.9. Add Google Fonts (Poppins + Inconsolata) per shared-ui v0.5.0+ theme. Adapt to typeberry/lib breaking changes: flattened block namespaces, removed ServiceStatistics.onTransfersCount, private WorkExecResult constructor, and Block.Codec.View in StateTransition. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for fluffy-codec ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThis PR updates the codebase to use new top-level exports from the Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
index.html (1)
9-14: Consider self-hosting these fonts to avoid third-party runtime dependency.This works, but loading fonts from Google at runtime (Line 9–Line 14) can create privacy/compliance and CSP friction, and adds an external availability dependency. Prefer bundling/self-hosting (e.g., via local assets or
@fontsource) if your deployment has strict privacy/security requirements.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@index.html` around lines 9 - 14, Replace the external Google Fonts links (the <link> tags loading Poppins and Inconsolata) with a self-hosted solution: either install and import the fonts via a package like `@fontsource` (e.g., `@fontsource/poppins` and `@fontsource/inconsolata`) or add the font files to your assets and reference them via local CSS `@font-face` rules; update the HTML to remove the Google CDN <link> tags and ensure your CSS uses the same font-family names (Poppins, Inconsolata) so existing styles continue to work, and verify CSP and build pipeline include the new font files.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@index.html`:
- Around line 9-14: Replace the external Google Fonts links (the <link> tags
loading Poppins and Inconsolata) with a self-hosted solution: either install and
import the fonts via a package like `@fontsource` (e.g., `@fontsource/poppins` and
`@fontsource/inconsolata`) or add the font files to your assets and reference them
via local CSS `@font-face` rules; update the HTML to remove the Google CDN <link>
tags and ensure your CSS uses the same font-family names (Poppins, Inconsolata)
so existing styles continue to work, and verify CSP and build pipeline include
the new font files.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4bd2dc1d-310d-4334-86a2-415d1e10dcdb
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (28)
biome.jsoncindex.htmlpackage.jsonsrc/components/constants.tssrc/components/examples/objects/availabilityAssurance.tssrc/components/examples/objects/credential.tssrc/components/examples/objects/culprit.tssrc/components/examples/objects/disputesExtrinsic.tssrc/components/examples/objects/fault.tssrc/components/examples/objects/importSpec.tssrc/components/examples/objects/judgement.tssrc/components/examples/objects/preimage.tssrc/components/examples/objects/refineContext.tssrc/components/examples/objects/reportGuarantee.tssrc/components/examples/objects/signedTicket.tssrc/components/examples/objects/ticket.tssrc/components/examples/objects/verdict.tssrc/components/examples/objects/workExecResult.tssrc/components/examples/objects/workItem.tssrc/components/examples/objects/workItemExtrinsicSpec.tssrc/components/examples/objects/workPackage.tssrc/components/examples/objects/workPackageSpec.tssrc/components/examples/objects/workReport.tssrc/components/examples/objects/workResult.tssrc/components/examples/state/recentBlocks.tssrc/components/examples/state/safroleData.tssrc/components/examples/state/statistics.tssrc/components/examples/stf/transition.ts
💤 Files with no reviewable changes (1)
- src/components/examples/state/statistics.ts
Summary
@typeberry/libbreaking changes: flattened block sub-namespaces (block.disputes.X→block.X), removedServiceStatistics.onTransfersCount, privateWorkExecResultconstructor → factory method,StateTransitionnow expectsBlock.Codec.ViewTest plan
npm run buildpasses (tsc + vite)npm run qa(biome ci) clean🤖 Generated with Claude Code