Skip to content

Commit 7749511

Browse files
committed
style: apply dart format and trim trailing blank lines
Output of the pre-push `dart format` / formatting hooks, which collapse a few single-argument calls onto one line and drop blank lines left behind when tests moved to `idle_reaper.test.ts`. No behaviour or logic change.
1 parent d37560c commit 7749511

10 files changed

Lines changed: 20 additions & 36 deletions

File tree

app/lib/app/router.dart

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,10 +56,7 @@ final routerProvider = Provider<GoRouter>((ref) {
5656
path: 'settings',
5757
builder: (_, _) => const SettingsScreen(),
5858
),
59-
GoRoute(
60-
path: 'closed',
61-
builder: (_, _) => const ClosedScreen(),
62-
),
59+
GoRoute(path: 'closed', builder: (_, _) => const ClosedScreen()),
6360
GoRoute(
6461
path: 'ports',
6562
builder: (_, s) =>

app/lib/desktop/chat/closed_sidebar_view.dart

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,7 @@ class ClosedSidebarView extends ConsumerStatefulWidget {
2828
const ClosedSidebarView({super.key});
2929

3030
@override
31-
ConsumerState<ClosedSidebarView> createState() =>
32-
_ClosedSidebarViewState();
31+
ConsumerState<ClosedSidebarView> createState() => _ClosedSidebarViewState();
3332
}
3433

3534
class _ClosedSidebarViewState extends ConsumerState<ClosedSidebarView> {

app/lib/desktop/chat/desktop_sidebar.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1229,8 +1229,8 @@ class _Footer extends ConsumerWidget {
12291229
color: showClosed ? theme.colorScheme.primary : null,
12301230
visualDensity: VisualDensity.compact,
12311231
constraints: const BoxConstraints(minWidth: 32, minHeight: 32),
1232-
onPressed: () => ref.read(sidebarClosedProvider.notifier).state =
1233-
!showClosed,
1232+
onPressed: () =>
1233+
ref.read(sidebarClosedProvider.notifier).state = !showClosed,
12341234
),
12351235
if (onOpenSettings != null)
12361236
IconButton(

app/lib/desktop/chat/panes/pane_header.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -204,9 +204,7 @@ class SessionActionsMenu extends ConsumerWidget {
204204
workspace.unbindSession(sessionId);
205205
} catch (e) {
206206
if (messenger.mounted) {
207-
messenger.showSnackBar(
208-
SnackBar(content: Text('Could not close: $e')),
209-
);
207+
messenger.showSnackBar(SnackBar(content: Text('Could not close: $e')));
210208
}
211209
}
212210
}

app/lib/ui/home/session_tile.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -175,9 +175,7 @@ class SessionTile extends ConsumerWidget {
175175
final confirmed = await _confirmQuit(context);
176176
if (!confirmed) return false;
177177
try {
178-
await ref
179-
.read(storeControllerProvider.notifier)
180-
.closeSession(session.id);
178+
await ref.read(storeControllerProvider.notifier).closeSession(session.id);
181179
return true;
182180
} catch (e) {
183181
messenger.showSnackBar(SnackBar(content: Text('Could not quit: $e')));

app/test/desktop/chat/closed_sidebar_view_test.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,7 @@ Future<_ClosedConn> _pumpClosed(
9696
overrides: [
9797
connectionControllerProvider.overrideWith((_) => conn),
9898
reposProvider.overrideWithValue(ReposState([_repo()])),
99-
sidebarClosedProvider.overrideWith(
100-
(_) => true,
101-
), // start in closed view
99+
sidebarClosedProvider.overrideWith((_) => true), // start in closed view
102100
],
103101
);
104102
addTearDown(container.dispose);

app/test/desktop/chat/pr_bar_test.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -618,9 +618,7 @@ void main() {
618618
expect(find.textContaining('Delete the local branch'), findsNothing);
619619
});
620620

621-
testWidgets('says sessions are closed, after the removal', (
622-
tester,
623-
) async {
621+
testWidgets('says sessions are closed, after the removal', (tester) async {
624622
// The server removes the worktree first and *closes* live sessions
625623
// (SPEC-29) rather than stopping them; the dialog claimed the reverse order
626624
// and the wrong verb.

app/test/desktop/chat/selected_session_test.dart

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -136,9 +136,7 @@ void main() {
136136
tree: WorkspaceController.seedWorkspace(),
137137
);
138138

139-
testWidgets('on a board the tab ✕ unpins and never closes', (
140-
tester,
141-
) async {
139+
testWidgets('on a board the tab ✕ unpins and never closes', (tester) async {
142140
final (conn, groups) = await runClose(tester, board(), (ref) {
143141
final state = ref.read(workspaceControllerProvider);
144142
final tab = activeTab(state)!;
@@ -153,17 +151,18 @@ void main() {
153151
);
154152
});
155153

156-
testWidgets('in a worktree group the tab ✕ closes the session (unchanged)', (
157-
tester,
158-
) async {
159-
final (conn, _) = await runClose(tester, worktree(), (ref) {
160-
final state = ref.read(workspaceControllerProvider);
161-
final tab = activeTab(state)!;
162-
closeTabAndSession(ref, state.activeSplitId, tab.id, tab.sessionId);
163-
});
154+
testWidgets(
155+
'in a worktree group the tab ✕ closes the session (unchanged)',
156+
(tester) async {
157+
final (conn, _) = await runClose(tester, worktree(), (ref) {
158+
final state = ref.read(workspaceControllerProvider);
159+
final tab = activeTab(state)!;
160+
closeTabAndSession(ref, state.activeSplitId, tab.id, tab.sessionId);
161+
});
164162

165-
expect(conn.sent.where((m) => m['kind'] == 'session.close').length, 1);
166-
});
163+
expect(conn.sent.where((m) => m['kind'] == 'session.close').length, 1);
164+
},
165+
);
167166

168167
testWidgets('⌘⇧W agrees with the ✕ on a board (one shared path)', (
169168
tester,

server/src/adapters/acp.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1574,4 +1574,3 @@ test("start() leaves the agent's model alone when it already matches, or is unkn
15741574
assert.ok(statuses.includes("idle"), "the session still started");
15751575
await a2.kill();
15761576
});
1577-

server/src/manager.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2510,5 +2510,3 @@ test("closeSession reaps even when the agent's close() never settles", async ()
25102510
store.close();
25112511
}
25122512
});
2513-
2514-

0 commit comments

Comments
 (0)