Skip to content

chore(pierre): upgrade @pierre/diffs to 1.1.19#214

Merged
benvinegar merged 1 commit intomainfrom
chore/upgrade-pierre-diffs-1.1.19
Apr 30, 2026
Merged

chore(pierre): upgrade @pierre/diffs to 1.1.19#214
benvinegar merged 1 commit intomainfrom
chore/upgrade-pierre-diffs-1.1.19

Conversation

@benvinegar
Copy link
Copy Markdown
Member

Summary

  • upgrade @pierre/diffs from ^1.1.0 to ^1.1.19
  • update Hunk's Pierre render options to satisfy the newer RenderDiffOptions contract
  • verify the current low-level Pierre integration still typechecks and passes targeted diff tests

What changed

  • bumped @pierre/diffs in package.json
  • refreshed bun.lock
  • added useTokenTransformer: false and maxLineDiffLength: 10_000 in src/ui/diff/pierre.ts

Notes

  • existing imports in src/ui/diff/pierre.ts, src/core/loaders.ts, src/opentui/index.ts, and test helpers remained valid
  • Hunk's flattening pipeline already matches the newer Pierre output assumptions we care about here, including direct color: styles and data-diff-span emphasis markers

Verification

  • bun run typecheck
  • bun test src/ui/diff/pierre.test.ts src/ui/diff/reviewRenderPlan.test.ts src/ui/lib/hunks.test.ts src/ui/lib/ui-lib.test.ts

This PR description was generated by Pi using GPT-5 Codex

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 29, 2026

Greptile Summary

This PR bumps @pierre/diffs from ^1.1.0 to ^1.1.19 (along with its @pierre/theme transitive dependency from 0.0.22 to 0.0.28) and adds two new fields — useTokenTransformer: false and maxLineDiffLength: 10_000 — to the shared Pierre render options to satisfy the updated RenderDiffOptions contract. The existing flattenHighlightedLine logic already handles the direct color:#... style output that useTokenTransformer: false produces, so the integration remains consistent.

Confidence Score: 5/5

Safe to merge — targeted dependency bump with a minimal, correct options change.

The change is a straightforward version bump. The two new option fields are added symmetrically to both themes, and the existing HAST flattening code already handles the direct color:#... output produced by useTokenTransformer: false. No logic, interface, or behavioural changes outside of the render config.

No files require special attention.

Important Files Changed

Filename Overview
src/ui/diff/pierre.ts Adds useTokenTransformer: false and maxLineDiffLength: 10_000 to both light and dark render option objects to satisfy the @pierre/diffs@1.1.19 RenderDiffOptions contract; no logic changes.
package.json Version range for @pierre/diffs bumped from ^1.1.0 to ^1.1.19.
bun.lock Lock file updated to resolve @pierre/diffs@1.1.19 and its transitive dependency @pierre/theme@0.0.28.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["loadHighlightedDiff()"] --> B["prepareHighlighter()\n(shiki-wasm)"]
    A --> C["renderDiffWithHighlighter()\n@pierre/diffs 1.1.19"]
    C -->|"useTokenTransformer: false\nmaxLineDiffLength: 10_000"| D["HAST output\n(direct color:# styles)"]
    D --> E["aliasHighlightedContextLines()"]
    E --> F["flattenHighlightedLine()\n(handles CSS var OR direct color)"]
    F --> G["buildSplitRows / buildStackRows\n→ RenderSpan[]"]
Loading

Reviews (1): Last reviewed commit: "chore(pierre): upgrade @pierre/diffs to ..." | Re-trigger Greptile

@benvinegar benvinegar merged commit b516dc8 into main Apr 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant