format

[Kitenite]

• testimonials (testimonials #56)
• update deps (update deps #57)
• Bump three from 0.180.0 to 0.181.0 (Bump three from 0.180.0 to 0.181.0 #9)
• Bump @types/three from 0.180.0 to 0.181.0 (Bump @types/three from 0.180.0 to 0.181.0 #11)
• downgrade node-pty (downgrade node-pty #61)
• open new window (open new window #59)
• Fix modal (Fix modal #63)
• run format

Description

Related Issues

Type of Change

• Bug fix
• New feature
• Documentation
• Refactor
• Other (please describe):

Testing

Screenshots (if applicable)

Additional Notes

Summary by CodeRabbit

• Style

  • Code formatting and indentation adjustments across multiple components and configuration files.

• Refactor

  • Adjusted placeholder state rendering logic to only display when specific conditions are met.

[coderabbitai]

Walkthrough

This PR reformats code across the monorepo by adjusting line breaks, indentation, and whitespace in TypeScript, TSX, and JSON files. Most changes are cosmetic; one control-flow adjustment in NewLayoutMain.tsx removes a loading check from the placeholder rendering condition.

Changes

Cohort / File(s)	Summary
Package.json trailing newlines apps/desktop/package.json, packages/models/package.json, packages/ui/package.json	Added/restored trailing newlines at end of files; no content changes.
Desktop app worktree error logging apps/desktop/src/main/lib/workspace-ipcs.ts, apps/desktop/src/main/lib/worktree-manager.ts	Reformatted log and error messages from single-line to multi-line strings; no functional or error-handling changes.
Desktop app layout components apps/desktop/src/renderer/screens/main/components/NewLayout/NewLayoutMain.tsx, apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/BasicWorktreeContent.tsx, apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/TaskWorktreeContent.tsx, apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/WorktreeTabButton.tsx	Adjusted JSX and conditional rendering formatting across multiple lines; removed loading check from placeholder rendering gate in NewLayoutMain.tsx.
Desktop app task/plan components apps/desktop/src/renderer/screens/main/components/PlanView/TaskCard.tsx, apps/desktop/src/renderer/screens/main/components/PlanView/TaskPage.tsx	Reformatted conditionals and string literals across multiple lines; no logic changes.
Desktop app sidebar component apps/desktop/src/renderer/screens/main/components/Sidebar/Sidebar.tsx	Expanded selectedWorktreeId prop across multiple lines preserving nullish coalescing logic.
Website testimonials component apps/website/src/app/components/TestimonialsSection/TestimonialsSection.tsx	Removed empty line and added line breaks around testimonial.title; no behavior changes.
Config files conductor.json, packages/ui/src/components/context-menu.tsx	JSON script object indentation adjustment; JSX return value reformatted to multi-line.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

• Special attention: apps/desktop/src/renderer/screens/main/components/NewLayout/NewLayoutMain.tsx — the loading check removal from the placeholder rendering condition changes control flow and should be verified for intent.

Possibly related PRs

• PR #56 — Modifies TestimonialsSection.tsx; this PR makes formatting adjustments to the same component.
• PR #48 — Modifies NewLayoutMain.tsx and worktree enrichment logic; overlaps with the control-flow change in this PR.
• PR #52 — Touches workspace-ipcs.ts and worktree-manager.ts where this PR reformats logging; related at the worktree PR handling level.

Poem

🐰 Lines reflow like morning dew,
Indents align, both old and new,
A loading check takes its leave,
Whitespace trimmed—no need to grieve,
Code now gleams, so neat, so true!

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description lists commit titles but contains no substantive content in any of the required template sections—all sections remain empty placeholders.	Fill in the Description section with a clear explanation of changes, select the Type of Change, and add any relevant details in Testing or Additional Notes sections.
Docstring Coverage	⚠️ Warning	Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check	❓ Inconclusive	The title 'format' is vague and generic, providing no meaningful information about the actual changes in the changeset.	Use a more descriptive title that summarizes the primary change, such as 'Format code across desktop, website, and packages' or 'Apply code formatting and fix modal issues'.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch format

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4492b5e and d7228bf.

📒 Files selected for processing (15)

• apps/desktop/package.json (1 hunks)
• apps/desktop/src/main/lib/workspace-ipcs.ts (1 hunks)
• apps/desktop/src/main/lib/worktree-manager.ts (4 hunks)
• apps/desktop/src/renderer/screens/main/components/NewLayout/NewLayoutMain.tsx (2 hunks)
• apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/BasicWorktreeContent.tsx (2 hunks)
• apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/TaskWorktreeContent.tsx (1 hunks)
• apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/WorktreeTabButton.tsx (1 hunks)
• apps/desktop/src/renderer/screens/main/components/PlanView/TaskCard.tsx (2 hunks)
• apps/desktop/src/renderer/screens/main/components/PlanView/TaskPage.tsx (4 hunks)
• apps/desktop/src/renderer/screens/main/components/Sidebar/Sidebar.tsx (1 hunks)
• apps/website/src/app/components/TestimonialsSection/TestimonialsSection.tsx (2 hunks)
• conductor.json (1 hunks)
• packages/models/package.json (1 hunks)
• packages/ui/package.json (1 hunks)
• packages/ui/src/components/context-menu.tsx (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/WorktreeTabButton.tsx (1)

apps/desktop/src/renderer/screens/main/components/NewLayout/StatusIndicator.tsx (1)

• StatusIndicator (53-113)

🪛 GitHub Actions: CI

apps/desktop/src/renderer/screens/main/components/PlanView/TaskPage.tsx

[error] 179-179: TS2554: Expected 2 arguments, but got 0.

🔇 Additional comments (19)

apps/desktop/package.json (1)

82-82: Trailing newline housekeeping. Clean formatting alignment with project conventions.

apps/website/src/app/components/TestimonialsSection/TestimonialsSection.tsx (1)

85-87: JSX formatting: multi-line title wrapping. Consistent line-breaking for improved readability in testimonial renders. No logic or rendering impact.

Also applies to: 139-141

apps/desktop/src/renderer/screens/main/components/PlanView/TaskCard.tsx (1)

69-72: Formatting refactors: condition line-wrapping and escape-sequence clarity. Both changes preserve logic while improving readability. Escape order (backslashes first) remains correct for safe shell invocation.

Also applies to: 88-90

apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/TaskWorktreeContent.tsx (1)

65-67: JSX formatting: tabs count display wrapped across lines. Improved readability; rendering behavior unchanged.

apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/BasicWorktreeContent.tsx (1)

17-19: JSX formatting: multi-line element wrapping. Title and tabs display elements reformatted for consistency. No rendering or logic impact.

Also applies to: 28-30

packages/models/package.json (1)

13-13: Trailing newline alignment. Consistent formatting across package manifests.

conductor.json (1)

2-4: JSON indentation cleanup. Scripts object reformatted without structural changes.

packages/ui/package.json (1)

58-58: Trailing newline alignment. Consistent manifest formatting across packages.

apps/desktop/src/renderer/screens/main/components/NewLayout/TaskTabs/WorktreeTabButton.tsx (1)

41-42: LGTM! Formatting change only.

The conditional rendering logic remains identical; only the line wrapping has changed.

packages/ui/src/components/context-menu.tsx (1)

10-16: LGTM! Formatting change only.

The return statement has been reformatted to multi-line JSX with no changes to attributes or behavior.

apps/desktop/src/renderer/screens/main/components/PlanView/TaskPage.tsx (4)

65-65: LGTM! Formatting change only.

The state initialization has been consolidated to a single line without altering behavior.

---

146-149: LGTM! Formatting change only.

The conditional expression has been reformatted across multiple lines with no behavioral change.

---

358-360: LGTM! Formatting change only.

The className expression has been reformatted across multiple lines without altering the conditional logic.

---

386-386: LGTM! Formatting change only.

The fallback URL expression has been consolidated to a single line without changing behavior.

apps/desktop/src/renderer/screens/main/components/Sidebar/Sidebar.tsx (1)

323-325: LGTM! Formatting change only.

The prop expression has been reformatted across multiple lines while preserving the identical nullish coalescing logic.

apps/desktop/src/main/lib/workspace-ipcs.ts (1)

561-563: LGTM! Formatting change only.

The console.log statement has been reformatted across multiple lines without altering the logged content.

apps/desktop/src/renderer/screens/main/components/NewLayout/NewLayoutMain.tsx (2)

244-249: LGTM! Formatting change only.

The object literal has been reformatted across multiple lines without altering the property values or structure.

---

1027-1031: LGTM! Formatting change only.

The conditional expression has been reformatted across multiple lines. The logic remains unchanged: PlaceholderState is rendered when loading, or when there's an error, or when required workspace/tab/worktree data is missing.

apps/desktop/src/main/lib/worktree-manager.ts (1)

727-728: LGTM! Formatting changes only.

Error messages and conditional expressions have been reformatted across multiple lines for improved readability. No behavioral changes to error handling or control flow.

Also applies to: 768-771, 775-776, 887-888, 938-939

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}