Skip to content

Fix prettier formatting for case status page#5850

Merged
beastoin merged 2 commits intomainfrom
fix/case-status-prettier
Mar 20, 2026
Merged

Fix prettier formatting for case status page#5850
beastoin merged 2 commits intomainfrom
fix/case-status-prettier

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

Summary

Run prettier on web/frontend/src/app/case/[ref]/page.tsx to fix CI lint failures from PR #5847.

Test plan

  • npx prettier --check passes
  • npx eslint passes

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 20, 2026

Greptile Summary

This PR applies automated Prettier formatting to web/frontend/src/app/case/[ref]/page.tsx to fix CI lint failures introduced in PR #5847. No logic, behaviour, or API surface was changed.

Key formatting changes:

  • Long object literals and JSX props expanded to multi-line to respect the configured print-width
  • Tailwind CSS utility classes reordered to follow the configured prettier-plugin-tailwindcss canonical sort order (e.g. flex min-h-screen items-center justify-center instead of min-h-screen … flex items-center justify-center)
  • Trailing commas added to function arguments and object entries to match the "trailingComma": "all" Prettier config
  • No functional, type, or routing changes of any kind

Confidence Score: 5/5

  • This PR is safe to merge — it contains only mechanical Prettier formatting changes with zero logic impact.
  • Every diff hunk is a pure whitespace/line-wrapping/comma/class-order change produced by the Prettier formatter. No logic, control flow, types, API calls, or component behaviour were altered. The change is fully automated and deterministic.
  • No files require special attention.

Important Files Changed

Filename Overview
web/frontend/src/app/case/[ref]/page.tsx Pure Prettier reformatting: long lines split, Tailwind classes reordered alphabetically, trailing commas added, JSX attributes expanded to multi-line — no logic or behaviour changes.

Sequence Diagram

sequenceDiagram
    participant Browser
    participant CaseStatusPage as CaseStatusPage (Server Component)
    participant API as API (api.omi.me)

    Browser->>CaseStatusPage: GET /case/[ref]
    CaseStatusPage->>CaseStatusPage: Validate ref format (FU-[hex]{6-12})
    alt Invalid format
        CaseStatusPage-->>Browser: notFound() → 404
    else Valid format
        CaseStatusPage->>API: GET /v1/fair-use/case/{ref}/status (10s timeout, no-store)
        alt 404
            API-->>CaseStatusPage: 404
            CaseStatusPage-->>Browser: "Case Not Found" UI
        else Non-2xx
            API-->>CaseStatusPage: error status
            CaseStatusPage-->>Browser: "Something Went Wrong" UI
        else 200 OK
            API-->>CaseStatusPage: CaseStatus JSON
            CaseStatusPage->>CaseStatusPage: safeEmail(), daysSince(), STAGE_META lookup
            CaseStatusPage-->>Browser: Case Status UI (stage badge, dates, message)
        end
    end
Loading

Last reviewed commit: "Fix prettier formatt..."

Remove unused beforeEach/afterEach imports, suppress no-undef for
globalThis, run prettier on test file.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@beastoin beastoin merged commit 38e1f29 into main Mar 20, 2026
7 checks passed
@beastoin beastoin deleted the fix/case-status-prettier branch March 20, 2026 03:57
@beastoin
Copy link
Copy Markdown
Collaborator Author

lgtm

kodjima33 pushed a commit that referenced this pull request Mar 20, 2026
* Fix prettier formatting for case status page

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix prettier formatting and lint errors in test file

Remove unused beforeEach/afterEach imports, suppress no-undef for
globalThis, run prettier on test file.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
* Fix prettier formatting for case status page

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix prettier formatting and lint errors in test file

Remove unused beforeEach/afterEach imports, suppress no-undef for
globalThis, run prettier on test file.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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