Skip to content

Commit 1d7ea48

Browse files
schpetschpet
committed
Add --all-teams flag to issue list command
Adds support for listing issues across all teams in a workspace using the --all-teams flag, addressing GitHub issue #140. Changes: - Add --all-teams option to issue list command - Update fetchIssuesForState to accept optional teamKey parameter - Add validation preventing simultaneous use of --all-teams and --team - Add test with deterministic timestamps using mocked Date - Update documentation in skills/linear-cli/references/issue.md The test now uses a fixed base date and mocked Date constructor to ensure deterministic timestamp output (e.g., '1 hour ago' instead of '759 days ago'), making snapshots reliable across different test runs.
1 parent f8efda7 commit 1d7ea48

5 files changed

Lines changed: 173 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: 131 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,133 @@ 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).toISOString(), // 1 hour ago
59+
},
60+
{
61+
id: "issue-2",
62+
identifier: "FRONTEND-456",
63+
title: "Update user interface",
64+
priority: 2,
65+
estimate: 5,
66+
assignee: {
67+
initials: "AS",
68+
},
69+
state: {
70+
id: "state-2",
71+
name: "To Do",
72+
color: "#6b7280",
73+
},
74+
labels: {
75+
nodes: [
76+
{
77+
id: "label-2",
78+
name: "feature",
79+
color: "#10b981",
80+
},
81+
],
82+
},
83+
updatedAt: new Date(baseDate.getTime() - 1000 * 60 * 60 * 24).toISOString(), // 1 day ago
84+
},
85+
{
86+
id: "issue-3",
87+
identifier: "SEC-789",
88+
title: "Security audit review",
89+
priority: 0,
90+
estimate: null,
91+
assignee: null,
92+
state: {
93+
id: "state-3",
94+
name: "Backlog",
95+
color: "#94a3b8",
96+
},
97+
labels: {
98+
nodes: [],
99+
},
100+
updatedAt: new Date(baseDate.getTime() - 1000 * 60 * 60 * 24 * 7).toISOString(), // 1 week ago
101+
},
102+
],
103+
pageInfo: {
104+
hasNextPage: false,
105+
endCursor: null,
106+
},
107+
},
108+
},
109+
},
110+
},
111+
])
112+
113+
try {
114+
await server.start()
115+
Deno.env.set("LINEAR_GRAPHQL_ENDPOINT", server.getEndpoint())
116+
Deno.env.set("LINEAR_API_KEY", "Bearer test-token")
117+
118+
// Mock the current date to make time calculations deterministic
119+
const originalDate = globalThis.Date
120+
// @ts-ignore: Mocking Date for testing
121+
globalThis.Date = class extends originalDate {
122+
constructor(...args: unknown[]) {
123+
if (args.length === 0) {
124+
super(baseDate.getTime())
125+
} else {
126+
// @ts-ignore: Mocking Date for testing
127+
super(...args)
128+
}
129+
}
130+
131+
static now() {
132+
return baseDate.getTime()
133+
}
134+
}
135+
136+
await listCommand.parse()
137+
138+
// Restore original Date
139+
globalThis.Date = originalDate
140+
} finally {
141+
await server.stop()
142+
Deno.env.delete("LINEAR_GRAPHQL_ENDPOINT")
143+
Deno.env.delete("LINEAR_API_KEY")
144+
}
145+
},
146+
})

0 commit comments

Comments
 (0)