Refresh Work lanes for mobile-created sessions

[arul28]

Summary

Describe the change.

What Changed

Key files and behaviors.

Validation

How you tested.

Risks

Anything to watch.

  Open in ADE  ·  ade/start-skill-then-why-getting-aef8fed2 branch  ·  PR #739

Summary by CodeRabbit

• Bug Fixes
  • Improved lane recovery in terminal work sessions so missing or outdated lane data is refreshed more reliably.
  • Prevented repeated refreshes when the same missing lane information is encountered.
  • Ensured lane recovery works correctly after switching projects or during hydration, helping sessions appear with the right lane details.

Greptile Summary

This PR updates Work session lane recovery to refresh lane data only when needed. The main changes are:

• Detects session laneId values that are missing from the current lane cache.
• Tracks refreshes by project and missing-lane signature to avoid duplicate recovery calls.
• Adds a regression test for sessions whose lane is absent while other cached lanes exist.

Confidence Score: 5/5

Safe to merge with minimal risk.

The change is localized to a memoized missing-lane signature and one recovery effect. Existing lightweight refresh options are preserved. A targeted test covers the new partial lane-cache case. No functional or security issues were identified in the changed paths.

No files require special attention.

T-Rex Logs

What T-Rex did

• T-Rex ran the Vitest test command for useWorkSessions and captured verbose output.
• T-Rex saved a proof log of the Vitest run documenting the command and results.
• T-Rex confirmed the relevant tests passed, including 'lightly refreshes lanes when Work sessions reference a lane missing from cached lane state' and 'does not retry lane recovery on every session refresh after a failure'.

_{Ran code and verified through T-Rex}

Important Files Changed

Filename	Overview
apps/desktop/src/renderer/components/terminals/useWorkSessions.ts	Replaces project-level lane recovery tracking with a missing-lane signature so refreshLanes runs when sessions reference lanes absent from cached lane state.
apps/desktop/src/renderer/components/terminals/useWorkSessions.test.ts	Adds coverage for refreshing lanes when cached lane state exists but does not include a session's lane.

Sequence Diagram

%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
participant Hook as useWorkSessions
participant Sessions as listSessionsCached
participant State as Lane state
participant Store as refreshLanes

Hook->>Sessions: load cached/current Work sessions
Sessions-->>Hook: sessions with laneId values
Hook->>State: read current lanes
Hook->>Hook: compute missingSessionLaneIdsSignature
alt signature is non-empty and not already refreshed
    Hook->>Store: refreshLanes(lightweight options)
    Store-->>State: update lane cache
else no missing lanes or duplicate signature
    Hook->>Hook: skip lane recovery refresh
end

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"}}}%%
sequenceDiagram
participant Hook as useWorkSessions
participant Sessions as listSessionsCached
participant State as Lane state
participant Store as refreshLanes

Hook->>Sessions: load cached/current Work sessions
Sessions-->>Hook: sessions with laneId values
Hook->>State: read current lanes
Hook->>Hook: compute missingSessionLaneIdsSignature
alt signature is non-empty and not already refreshed
    Hook->>Store: refreshLanes(lightweight options)
    Store-->>State: update lane cache
else no missing lanes or duplicate signature
    Hook->>Hook: skip lane recovery refresh
end

Loading

_{Reviews (1): Last reviewed commit: "fix: refresh work lanes for mobile-creat..." | Re-trigger Greptile}

[cursor]

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
ade	Ignored		Jul 8, 2026 8:20pm

[arul28]

@copilot review but do not make fixes

[coderabbitai]

📝 Walkthrough

Walkthrough

The lane-recovery logic in useWorkSessions was reworked to derive a missingSessionLaneIdsSignature from sessions referencing lanes absent from the cached lane store, replacing the prior project-based refresh key, and gates refreshLanes calls on signature changes. A new test verifies this behavior.

Changes

Lane recovery signature rework

Layer / File(s)	Summary
Missing-lane signature and recovery effect apps/desktop/src/renderer/components/terminals/useWorkSessions.ts	Replaces the project-based recovery ref with laneRecoveryRefreshKeyRef and a missingSessionLaneIdsSignature memo; the recovery effect now refreshes lanes only when the signature-based key changes, and resets the key on project hydration.
Missing-lane refresh test apps/desktop/src/renderer/components/terminals/useWorkSessions.test.ts	Adds a test confirming refreshLanesSpy runs a light refresh exactly once when a session references a lane missing from cached lane state.

Estimated code review effort: 2 (Simple) | ~12 minutes

Possibly related PRs

• arul28/ADE#219: Modifies refreshLanes to accept/merge the same lane snapshot include-flag options used by the updated recovery logic in this PR.

Suggested labels: desktop

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main fix: refreshing work lanes for sessions created on mobile when lane data is missing.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ade/start-skill-then-why-getting-aef8fed2

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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