Skip to content

style(desktop): eslint import-order fixes for the transcript-stash change - #367

Merged
Kyzcreig merged 1 commit into
mainfrom
style/stash-eslint-fixes
Jul 16, 2026
Merged

style(desktop): eslint import-order fixes for the transcript-stash change#367
Kyzcreig merged 1 commit into
mainfrom
style/stash-eslint-fixes

Conversation

@Kyzcreig

Copy link
Copy Markdown
Collaborator

Follow-up to #366: eslint --fix import ordering (perfectionist/sort-imports, sort-named-imports) + padding-line rules on the two files the stash change touched. No behavior change — stash tests 8/8 green, tsc clean, eslint 0 errors on the touched files.

@Kyzcreig
Kyzcreig enabled auto-merge (squash) July 16, 2026 07:09
@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown

Greptile Summary

This is a pure style follow-up to #366 that applies ESLint auto-fix results (perfectionist/sort-imports, sort-named-imports, and padding-line rules) to the two files touched by the transcript-stash change. There are no logic or behavior changes.

  • use-session-actions/index.ts: Relative imports (render-cache-hydration, transcript-stash) moved to after the absolute-path import block to satisfy perfectionist/sort-imports ordering; blank lines added around a handful of statements per padding-line rules.
  • transcript-stash.test.ts: Named imports reordered so stashedTranscriptCount comes before stashTranscript alphabetically.

Confidence Score: 5/5

Safe to merge — all changes are ESLint auto-fix output with no functional code touched.

Every change in this PR is a mechanical formatting adjustment: import reordering and blank-line insertion. No runtime paths, data structures, or test assertions were modified. The PR description confirms all tests pass and TypeScript is clean.

No files require special attention.

Important Files Changed

Filename Overview
apps/desktop/src/app/session/hooks/use-session-actions/index.ts Import ordering corrected (relative imports moved after absolute-path block) and blank-line padding added around several statements; no logic changes.
apps/desktop/src/app/transcript-stash.test.ts Named imports reordered alphabetically (stashedTranscriptCount before stashTranscript); no test logic changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[ESLint --fix run] --> B{Rules applied}
    B --> C[perfectionist/sort-imports\nRelative imports moved after absolute-path block]
    B --> D[sort-named-imports\nAlphabetical named-import reorder]
    B --> E[padding-line\nBlank lines added around statements]
    C --> F[use-session-actions/index.ts]
    D --> G[transcript-stash.test.ts]
    E --> F
Loading
%%{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[ESLint --fix run] --> B{Rules applied}
    B --> C[perfectionist/sort-imports\nRelative imports moved after absolute-path block]
    B --> D[sort-named-imports\nAlphabetical named-import reorder]
    B --> E[padding-line\nBlank lines added around statements]
    C --> F[use-session-actions/index.ts]
    D --> G[transcript-stash.test.ts]
    E --> F
Loading

Reviews (1): Last reviewed commit: "style(desktop): eslint --fix import orde..." | Re-trigger Greptile

@Kyzcreig
Kyzcreig merged commit d6e5994 into main Jul 16, 2026
24 checks passed
@Kyzcreig
Kyzcreig deleted the style/stash-eslint-fixes branch July 16, 2026 07:10
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