@marcomigozzi/trailhead 0.16.0 → 0.17.0

package/plugins/trailhead/tests/my-tickets.test.ts

@@ -0,0 +1,66 @@
+import { expect, test } from 'claude-code/testing'
+
+import { parseTickets } from '../hooks/mod/map-pane-view'
+import { elsewhereInProgress, myTicketsFromRows, parseLogin, scopeLine } from '../hooks/mod/my-tickets'
+
+const row = (number: number, names: string[], logins: string[] = ['me']) => ({
+  number,
+  title: `t${number}`,
+  url: `https://github.com/o/r/issues/${number}`,
+  labels: names.map(name => ({ name })),
+  assignees: logins.map(login => ({ login })),
+})
+
+test('parseLogin reads the viewer login, null on anything else', () => {
+  expect(parseLogin(JSON.stringify({ data: { viewer: { login: 'me' } } }))).toBe('me')
+  expect(parseLogin(JSON.stringify({ data: { viewer: { login: '' } } }))).toBeNull()
+  expect(parseLogin(JSON.stringify({ errors: [{}], data: { viewer: { login: 'me' } } }))).toBeNull()
+  expect(parseLogin('not json')).toBeNull()
+})
+
+test('myTicketsFromRows tags each ticket with its map or the whiteboard, and drops the rest', () => {
+  const mine = myTicketsFromRows([
+    row(5, ['trailhead:ticket', 'trailhead:build', 'trailhead:map-12']),
+    row(6, ['trailhead:ticket', 'trailhead:bug', 'trailhead:whiteboard']),
+    // no scope, contradictory scopes, and quarantined: never shown
+    row(7, ['trailhead:ticket', 'trailhead:build']),
+    row(8, ['trailhead:ticket', 'trailhead:map-1', 'trailhead:whiteboard']),
+    row(9, ['trailhead:ticket', 'trailhead:map-1', 'trailhead:map-2']),
+    row(10, ['trailhead:ticket', 'trailhead:map-1', 'trailhead:unverified']),
+  ])
+  expect(mine.map(t => [t.number, t.scope])).toEqual([
+    [5, { kind: 'map', number: 12 }],
+    [6, { kind: 'whiteboard' }],
+  ])
+  expect(mine[0]?.state).toBe('in-progress')
+})
+
+test('elsewhereInProgress keeps my in-progress tickets of the other scopes only', () => {
+  const mine = myTicketsFromRows([
+    row(1, ['trailhead:ticket', 'trailhead:map-12']),
+    row(2, ['trailhead:ticket', 'trailhead:map-30']),
+    row(3, ['trailhead:ticket', 'trailhead:whiteboard']),
+    // not in progress: paused, blocked, needs-you, UAT
+    row(4, ['trailhead:ticket', 'trailhead:map-30', 'trailhead:paused']),
+    row(5, ['trailhead:ticket', 'trailhead:map-30', 'trailhead:blocked']),
+    row(6, ['trailhead:ticket', 'trailhead:map-30', 'trailhead:needs-you']),
+    row(7, ['trailhead:ticket', 'trailhead:map-30', 'trailhead:uat']),
+  ])
+  expect(elsewhereInProgress(mine, { kind: 'map', number: 12 }, []).map(t => t.number)).toEqual([2, 3])
+  expect(elsewhereInProgress(mine, { kind: 'whiteboard' }, []).map(t => t.number)).toEqual([1, 2])
+  // a ticket the view already lists is never shown twice
+  const shown = parseTickets(JSON.stringify([row(2, ['trailhead:ticket', 'trailhead:map-12'])]))?.tickets ?? []
+  expect(elsewhereInProgress(mine, { kind: 'map', number: 12 }, shown).map(t => t.number)).toEqual([3])
+})
+
+test('scopeLine names the map by number and title, or the whiteboard, cut to the width', () => {
+  const [onMap, onBoard] = myTicketsFromRows([
+    row(1, ['trailhead:ticket', 'trailhead:map-30']),
+    row(2, ['trailhead:ticket', 'trailhead:whiteboard']),
+  ])
+  const maps = [{ number: 30, title: 'Billing rework', counts: null }]
+  expect(onMap && scopeLine(onMap, maps, 60)).toBe('on map #30 Billing rework')
+  expect(onMap && scopeLine(onMap, [], 60)).toBe('on map #30')
+  expect(onBoard && scopeLine(onBoard, maps, 60)).toBe('on whiteboard')
+  expect((onMap && scopeLine(onMap, maps, 16))?.length).toBeLessThanOrEqual(16)
+})

package/plugins/trailhead/tests/repo-pin.test.ts

@@ -112,12 +112,27 @@ test('repoFromGit keeps a repo with no origin remote unpinned, so the last pin s
   expect(nextPin(current, repo)).toBe(current)
 })
 
-test('checkoutArg hands the root over as one shell-safe word the helper decodes back', () => {
+const fromBase64url = (token: string): string => {
+  const bytes = Uint8Array.from(atob(token.replace(/-/g, '+').replace(/_/g, '/').padEnd(Math.ceil(token.length / 4) * 4, '=')), c => c.charCodeAt(0))
+  return new TextDecoder().decode(bytes)
+}
+
+test('checkoutArg hands the root and the repo over as one shell-safe word the helper decodes back', () => {
   for (const root of ['/repo', '/with space/r', "/it's/r", '/-dash/r', '/uni\u00e8\u4e2d/r', '/a"b$c`d']) {
     const arg = checkoutArg({ ...GH, root })
-    expect(arg).toMatch(/^--in=[A-Za-z0-9_-]+$/)
-    const token = arg.slice('--in='.length)
-    const bytes = Uint8Array.from(atob(token.replace(/-/g, '+').replace(/_/g, '/').padEnd(Math.ceil(token.length / 4) * 4, '=')), c => c.charCodeAt(0))
-    expect(new TextDecoder().decode(bytes)).toBe(root)
+    expect(arg).toMatch(/^--in=[A-Za-z0-9_-]+\.[A-Za-z0-9_-]+$/)
+    const [path, repo] = arg.slice('--in='.length).split('.')
+    expect(fromBase64url(path)).toBe(root)
+    expect(fromBase64url(repo)).toBe('github.com/o/r')
   }
 })
+
+test('checkoutArg carries host and a nested slug in the repo part', () => {
+  const arg = checkoutArg({ root: '/w/p', host: 'gitlab.com', slug: 'g/sub/r' })
+  expect(fromBase64url(arg.split('.')[1])).toBe('gitlab.com/g/sub/r')
+})
+
+test('checkoutArg carries a dotless host in the repo part', () => {
+  const arg = checkoutArg({ root: '/w/p', host: 'gitea', slug: 'owner/name' })
+  expect(fromBase64url(arg.split('.')[1])).toBe('gitea/owner/name')
+})

package/plugins/trailhead/types/index.d.ts

@@ -42,6 +42,11 @@ export type PaneTicket = {
   uatOf: UatParent | null
 }
 
+// Where one of my tickets lives: an open map by number, or the whiteboard.
+export type TicketScope = { kind: 'map'; number: number } | { kind: 'whiteboard' }
+// A ticket assigned to the gh login, tagged with its scope (the In progress group lists it outside that scope).
+export type MyTicket = PaneTicket & { scope: TicketScope }
+
 // A map's sub-issue counts: `open` still to do, `total` all of them.
 export type MapCounts = { open: number; total: number }
 // One open map of the repo, as the maps list shows it; `counts` null = unknown, drawn name-only.
@@ -113,6 +118,8 @@ declare module 'claude-code' {
       // The pane shows the dashboard (maps index + whiteboard) instead of a map.
       isDashboard: boolean
       dashboardPane: DashboardPaneData
+      // My open tickets across every map and the whiteboard (the last good read), for every view's In progress.
+      myTickets: readonly MyTicket[]
       pendingWork: number | null
       // Whether the pending press waits for the turn to end or has been sent.
       pendingPhase: 'queued' | 'starting' | null