Skip to content

Commit f032161

Browse files
schpetschpet
committed
Add --all-teams flag to issue list command
- Add --all-teams flag to allow viewing issues from all teams - Make teamKey optional in fetchIssuesForState when using --all-teams - Add deterministic tests with mocked Date to ensure snapshots are stable - Validate mutual exclusion between --all-teams and --team flags - Update documentation to reflect the new flag
1 parent 6df2efd commit f032161

5 files changed

Lines changed: 178 additions & 12 deletions

File tree

‎skills/linear-cli/references/issue.md‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,11 @@ Options:
7777
--all-states - Show issues from all states
7878
--assignee <assignee> - Filter by assignee (username)
7979
-A, --all-assignees - Show issues for all assignees
80-
-U, --unassigned - Show only unassigned issues
81-
--sort <sort> - Sort order (can also be set via LINEAR_ISSUE_SORT) (Values: "manual", "priority")
82-
--team <team> - Team to list issues for (if not your default team)
83-
--project <project> - Filter by project name
80+
-U, --unassigned - Show only unassigned issues
81+
--sort <sort> - Sort order (can also be set via LINEAR_ISSUE_SORT) (Values: "manual", "priority")
82+
--team <team> - Team to list issues for (if not your default team)
83+
--all-teams - Show issues from all teams in the workspace
84+
--project <project> - Filter by project name
8485
--limit <limit> - Maximum number of issues to fetch (default: 50, use 0 for unlimited) (Default: 50)
8586
-w, --web - Open in web browser
8687
-a, --app - Open in Linear.app

‎src/commands/issue/issue-list.ts‎

Lines changed: 20 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ export const listCommand = new Command()
7575
"--team <team:string>",
7676
"Team to list issues for (if not your default team)",
7777
)
78+
.option(
79+
"--all-teams",
80+
"Show issues from all teams in the workspace",
81+
)
7882
.option(
7983
"--project <project:string>",
8084
"Filter by project name",
@@ -101,6 +105,7 @@ export const listCommand = new Command()
101105
app,
102106
allStates,
103107
team,
108+
allTeams,
104109
project,
105110
limit,
106111
pager,
@@ -131,6 +136,10 @@ export const listCommand = new Command()
131136
throw new ValidationError("Cannot use --all-states with --state flag")
132137
}
133138

139+
if (allTeams && team) {
140+
throw new ValidationError("Cannot use --all-teams with --team flag")
141+
}
142+
134143
const sort = sortFlag ||
135144
getOption("issue_sort") as "manual" | "priority" | undefined
136145
if (!sort) {
@@ -143,11 +152,17 @@ export const listCommand = new Command()
143152
`Sort must be one of: ${SortType.values().join(", ")}`,
144153
)
145154
}
146-
const teamKey = team || getTeamKey()
147-
if (!teamKey) {
148-
throw new ValidationError(
149-
"Could not determine team key from directory name or team flag",
150-
)
155+
156+
let teamKey: string | undefined
157+
if (allTeams) {
158+
teamKey = undefined
159+
} else {
160+
teamKey = team || getTeamKey()
161+
if (!teamKey) {
162+
throw new ValidationError(
163+
"Could not determine team key from directory name or team flag",
164+
)
165+
}
151166
}
152167

153168
let projectId: string | undefined

‎src/utils/linear.ts‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,7 @@ export async function fetchParentIssueData(parentId: string): Promise<
389389
}
390390

391391
export async function fetchIssuesForState(
392-
teamKey: string,
392+
teamKey: string | undefined,
393393
state: string[] | undefined,
394394
assignee?: string,
395395
unassigned = false,
@@ -410,8 +410,10 @@ export async function fetchIssuesForState(
410410
)
411411
}
412412

413-
const filter: IssueFilter = {
414-
team: { key: { eq: teamKey } },
413+
const filter: IssueFilter = {}
414+
415+
if (teamKey) {
416+
filter.team = { key: { eq: teamKey } }
415417
}
416418

417419
if (state) {

‎test/commands/issue/__snapshots__/issue-list.test.ts.snap‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ Options:
2020
-U, --unassigned - Show only unassigned issues
2121
--sort <sort> - Sort order (can also be set via LINEAR_ISSUE_SORT) (Values: \\x1b[32m"manual"\\x1b[39m, \\x1b[32m"priority"\\x1b[39m)
2222
--team <team> - Team to list issues for (if not your default team)
23+
--all-teams - Show issues from all teams in the workspace
2324
--project <project> - Filter by project name
2425
--limit <limit> - Maximum number of issues to fetch (default: 50, use 0 for unlimited) (Default: \\x1b[33m50\\x1b[39m)
2526
-w, --web - Open in web browser
@@ -30,3 +31,14 @@ Options:
3031
stderr:
3132
""
3233
`;
34+
35+
snapshot[`Issue List Command - All Teams 1`] = `
36+
stdout:
37+
"◌ ID TITLE LABELS E STATE UPDATED
38+
⚠⚠⚠ BACKEND-123 Fix authentication bug bug 3 In Progress 1 hour ago
39+
▄▆█ FRONTEND-456 Update user interface feature 5 To Do 1 day ago
40+
--- SEC-789 Security audit review - Backlog 7 days ago
41+
"
42+
stderr:
43+
""
44+
`;

‎test/commands/issue/issue-list.test.ts‎

Lines changed: 136 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import { snapshotTest } from "@cliffy/testing"
22
import { listCommand } from "../../../src/commands/issue/issue-list.ts"
33
import { commonDenoArgs } from "../../utils/test-helpers.ts"
4+
import { MockLinearServer } from "../../utils/mock_linear_server.ts"
45

56
// Test help output
67
await snapshotTest({
@@ -13,3 +14,138 @@ await snapshotTest({
1314
await listCommand.parse()
1415
},
1516
})
17+
18+
// Test --all-teams flag
19+
await snapshotTest({
20+
name: "Issue List Command - All Teams",
21+
meta: import.meta,
22+
colors: false,
23+
args: ["--all-teams", "--no-pager", "--sort", "manual"],
24+
denoArgs: commonDenoArgs,
25+
async fn() {
26+
// Use a fixed date for deterministic snapshot tests
27+
const baseDate = new Date("2024-01-15T12:00:00Z")
28+
const server = new MockLinearServer([
29+
{
30+
queryName: "GetIssuesForState",
31+
response: {
32+
data: {
33+
issues: {
34+
nodes: [
35+
{
36+
id: "issue-1",
37+
identifier: "BACKEND-123",
38+
title: "Fix authentication bug",
39+
priority: 1,
40+
estimate: 3,
41+
assignee: {
42+
initials: "JD",
43+
},
44+
state: {
45+
id: "state-1",
46+
name: "In Progress",
47+
color: "#3b82f6",
48+
},
49+
labels: {
50+
nodes: [
51+
{
52+
id: "label-1",
53+
name: "bug",
54+
color: "#ef4444",
55+
},
56+
],
57+
},
58+
updatedAt: new Date(baseDate.getTime() - 1000 * 60 * 60)
59+
.toISOString(), // 1 hour ago
60+
},
61+
{
62+
id: "issue-2",
63+
identifier: "FRONTEND-456",
64+
title: "Update user interface",
65+
priority: 2,
66+
estimate: 5,
67+
assignee: {
68+
initials: "AS",
69+
},
70+
state: {
71+
id: "state-2",
72+
name: "To Do",
73+
color: "#6b7280",
74+
},
75+
labels: {
76+
nodes: [
77+
{
78+
id: "label-2",
79+
name: "feature",
80+
color: "#10b981",
81+
},
82+
],
83+
},
84+
updatedAt: new Date(baseDate.getTime() - 1000 * 60 * 60 * 24)
85+
.toISOString(), // 1 day ago
86+
},
87+
{
88+
id: "issue-3",
89+
identifier: "SEC-789",
90+
title: "Security audit review",
91+
priority: 0,
92+
estimate: null,
93+
assignee: null,
94+
state: {
95+
id: "state-3",
96+
name: "Backlog",
97+
color: "#94a3b8",
98+
},
99+
labels: {
100+
nodes: [],
101+
},
102+
updatedAt: new Date(
103+
baseDate.getTime() - 1000 * 60 * 60 * 24 * 7,
104+
).toISOString(), // 1 week ago
105+
},
106+
],
107+
pageInfo: {
108+
hasNextPage: false,
109+
endCursor: null,
110+
},
111+
},
112+
},
113+
},
114+
},
115+
])
116+
117+
try {
118+
await server.start()
119+
Deno.env.set("LINEAR_GRAPHQL_ENDPOINT", server.getEndpoint())
120+
Deno.env.set("LINEAR_API_KEY", "Bearer test-token")
121+
122+
// Mock the current date to make time calculations deterministic
123+
const originalDate = globalThis.Date
124+
// @ts-ignore: Mocking Date for testing
125+
globalThis.Date = class extends originalDate {
126+
// deno-lint-ignore constructor-super
127+
constructor(...args: unknown[]) {
128+
if (args.length === 0) {
129+
super(baseDate.getTime())
130+
} else {
131+
// @ts-ignore: Mocking Date for testing
132+
super(...args)
133+
}
134+
}
135+
136+
static override now() {
137+
return baseDate.getTime()
138+
}
139+
}
140+
141+
await listCommand.parse()
142+
143+
// Restore original Date
144+
globalThis.Date = originalDate
145+
} finally {
146+
await server.stop()
147+
Deno.env.delete("LINEAR_GRAPHQL_ENDPOINT")
148+
Deno.env.delete("LINEAR_API_KEY")
149+
}
150+
},
151+
})

0 commit comments

Comments
 (0)