@preventive/triage 1.0.0-alpha.49 → 1.0.0-alpha.50

package/server-managed/http.ts

@@ -135,8 +135,9 @@ import { mergeLinkGroups } from './link-reports.ts'
 import { backfillFindingIds, reportEntries } from '@preventive/report'
 import type { IncomingMessage, ServerResponse } from 'node:http'
 import { pipeline } from 'node:stream/promises'
-import { type KeptStore, REPOSITORY_READ_KEPT_MS, codeHeadStore, codeTreeStore, historyChangeStore, historyPageStore, historySquashStore, historyTreeStore, lockfileReadStore,
-  npmDeltaStore, npmPublishTimeStore, npmVersionStore, ownKeptPrunes, pullDependencyStore, repositoryReadStore, reviewBaselineStore, riskLineStore, scanReadStore } from './npm-kept.ts'
+import { type KeptStore, REPOSITORY_READ_KEPT_MS, codeHeadStore, codeTreeStore, commitPullStore, historyChangeStore, historyPageStore, historySquashStore, historyTreeStore,
+  lockfileReadStore,
+  npmDeltaStore, npmPublishTimeStore, npmVersionStore, ownKeptPrunes, pullDependencyStore, pullListStore, repositoryReadStore, reviewBaselineStore, riskLineStore, scanReadStore } from './npm-kept.ts'
 import { plainObject } from './npm-manifest.ts'
 import { type NpmDelta, keptNpmDeltas, npmDelta } from './npm-delta.ts'
 import { ADVISORIES_TIMEOUT_MS, fetchBundleAdvisories } from './bundle-advisories.ts'
@@ -190,7 +191,7 @@ import { isExactVersion } from '@preventive/upstream/semver.js'
 import { VIEW_AS_PATH, VIEW_ONLY_ERROR, clearViewCookie, endViewSession, runViewing, startViewSession, viewToken } from './view-as.ts'
 import type { ActivityContext, ActivityInput } from './activity.ts'
 import { acceptsReportMetadata } from './report-response.ts'
-import { TeamReportsError, loadTeamAnnotations, loadTeamReportsResponse, loadTeamTreesResponse, recheckTeam, teamFindingIds, teamSnapshot, teamSourcePaths, teamWorkspaceFindingIds } from './team-reports.ts'
+import { TeamReportsError, type TeamResponsePart, loadTeamAnnotations, loadTeamReportsResponse, recheckTeam, teamFindingIds, teamResponsePart, teamSnapshot, teamSourcePaths, teamWorkspaceFindingIds } from './team-reports.ts'
 import { FINDING_CATALOG_PAGE_BYTES, FINDING_CATALOG_PAGE_COUNT, MAX_REPORT_QUERY_BYTES, MAX_REPORT_QUERY_COUNT } from './report-query.ts'
 import { lookupFixes, storedFixUrls } from './github-pulls.ts'
 import { lookupIssueLinks } from './github-issue-links.ts'
@@ -214,11 +215,11 @@ import { NpmPackageError, type NpmReader, type NpmVersionDocument, type NpmVersi
 import { loadNpmFiles, loadNpmPackageBody, loadNpmTarball } from './npm-loads.ts'
 import { type RepositoryCodeKept, loadRepositoryCodeBody, repositoryCodeFits, repositoryCodeKept } from './repository-code.ts'
 import { type ChangesStore, type CommitChanges, type CommitTree, TREES_READ, changesAsked, changesWithin, directoryFiles, repositoryChanges } from './repository-changes.ts'
-import { COAUTHORS_READ } from './commit-coauthors.ts'
+import { COAUTHORS_READ, coauthorsOf } from './commit-coauthors.ts'
 import { type AllHeads, type HistoryCommit, type HistoryPages, MAX_HISTORY_PAGE, historyFrom, historyStarts, openPullHeads,
   repositoryAllHeads, repositoryAllHistory, repositoryHistory, repositoryRefTips } from './repository-history.ts'
-import { type SquashedPull, type Squashes, squashedPulls } from './repository-squashes.ts'
-import { AGENTS_READ_VERSION, COMMITS_CURSOR, repositoryAgents } from './repository-agents.ts'
+import { type KnownMerges, type SquashedPull, type Squashes, squashedPulls } from './repository-squashes.ts'
+import { AGENTS_READ_VERSION, COMMITS_CURSOR, type CommitPulls, repositoryAgents } from './repository-agents.ts'
 import { ISSUES_CURSOR, ISSUES_READ_VERSION, repositoryIssues, repositoryItem } from './repository-issues.ts'
 import { type GithubAsked, type ListedLockfile, type LockfileRead, type LockfileStore, MAX_LOCKED, PULL_DEPENDENCIES_READ, type PullCommits, type PullDependencies,
   type PullDependencyStore, eachOf, lockfileFormat, pullDependenciesAt } from './pull-dependencies.ts'
@@ -232,6 +233,8 @@ import { commentReviews } from './pull-comment-reviews.ts'
 import { PULSE_WEEKS, PULSE_YEAR_VERSION, type PulseStats, StatsPending, commitStats, defaultHead, periodStats, pulseSince, repositoryActivity, repositoryLines,
   repositoryStats, withLines, yearCommits } from './repository-pulse.ts'
 import { type AgentMerges, agentCommits, agentMerges, directCommits, sameMerges } from './agent-merges.ts'
+import { PULLS_READ_VERSION, type PullList, pullsReadTo, readPulls } from './repository-pulls.ts'
+import { AGENT_MARKS_READ } from './app-agents.ts'
 import { releaseCommits } from './release-commits.ts'
 import { computeBundleDiff } from '../common/bundle-compare-diff.js'
 import type { ReviewChange, ReviewContext, ReviewInput, ReviewedTarballs } from './dependency-reviews.ts'
@@ -1882,7 +1885,7 @@ async function handleRepositoryHistory(res: ServerResponse, deps: ManagedHttpDep
     const listed = (body as { commits: HistoryCommit[] }).commits
     const none = { pulls: [] as SquashedPull[], pending: false }
     const [squashed, openPulls] = token && path === '' ? await Promise.all([
-      squashedPulls(repo, token, listed, historySquashes(deps, repoId), SQUASH_WAIT_MS).then(read => {
+      pulseMerges(deps, repoId).catch(quiet(null)).then(known => squashedPulls(repo, token, listed, historySquashes(deps, repoId), SQUASH_WAIT_MS, globalThis.fetch, known)).then(read => {
         if (read.behind) {
           const behind = read.behind.catch(quiet(null))
           deps.track(behind)
@@ -1897,10 +1900,11 @@ async function handleRepositoryHistory(res: ServerResponse, deps: ManagedHttpDep
     const current = await authorize()
     if (!current) return
     if (!sameRepository(current.repo, access.repo)) { sendJson(res, 409, { error: 'repository-changed' }); return }
-    // Each commit as the tab draws it, its co-authors with it: what Pulse
-    // counts of it (its lines, its author's time) not sent.
-    const shown = ({ sha, parents, subject, authorName, authorLogin, authorId, coauthors, committedAt }: HistoryCommit) =>
-      ({ sha, parents, subject, authorName, authorLogin, authorId: authorId ?? null, coauthors: coauthors ?? [], committedAt })
+    // Each commit as the tab draws it, its co-authors with it, told by the
+    // agents the app knows now (coauthorsOf): what Pulse counts of it (its
+    // lines, its author's time) and the addresses kept not sent.
+    const shown = (commit: HistoryCommit) => ({ sha: commit.sha, parents: commit.parents, subject: commit.subject, authorName: commit.authorName,
+      authorLogin: commit.authorLogin, authorId: commit.authorId ?? null, coauthors: coauthorsOf(commit), committedAt: commit.committedAt })
     sendJson(res, 200, await withAppUsers(deps.db, { ...body, commits: listed.map(shown),
       ...pulls.length > 0 ? { pulls: pulls.map(pull => ({ ...pull, commits: pull.commits.map(shown) })) } : {}, ...squashed.pending ? { pullsPending: true } : {},
       ...openPulls?.length ? { openPulls } : {} }))
@@ -2074,14 +2078,16 @@ async function keptAnswer<T extends object>(deps: ManagedHttpDeps, key: string,
 // telling each commit's co-authors (kept since they are read), and with
 // `graphql`, only those read with GraphQL, telling its lines and the accounts
 // its co-authors' addresses are (a page GitHub's REST API listed tells
-// neither); the rest read again.
-function historyPages(deps: ManagedHttpDeps, repoId: number, graphql = false): HistoryPages {
+// neither), and with `marks`, only those telling the marks agents left on
+// its commits by the rules they are read by now (app-agents.ts
+// AGENT_MARKS_READ), as the Agents tab reads them; the rest read again.
+function historyPages(deps: ManagedHttpDeps, repoId: number, graphql = false, marks = false): HistoryPages {
   const store = historyPageStore<HistoryCommit[]>(deps.db, deps.config.debug)
   const keyOf = (top: string) => `${repoId}\0${COAUTHORS_READ}\0${top}`
   const told = (tops: string[], kept: Map<string, { value: HistoryCommit[] }>) => new Map(tops.flatMap(top => {
     const page = kept.get(keyOf(top))?.value
-    return Array.isArray(page) && page.every(commit => Array.isArray(commit.coauthors)) && (!graphql || page.every(commit => commit.additions != null))
-      ? [[top, page] as const] : []
+    return Array.isArray(page) && page.every(commit => Array.isArray(commit.named)) && (!graphql || page.every(commit => commit.additions != null))
+      && (!marks || page.every(commit => commit.marked === AGENT_MARKS_READ)) ? [[top, page] as const] : []
   }))
   return {
     read: async tops => told(tops, await store.readMany(tops.map(keyOf))),
@@ -2093,6 +2099,73 @@ function historyPages(deps: ManagedHttpDeps, repoId: number, graphql = false): H
   }
 }
 
+// Whether a pull request brought each of repository `repoId`'s commits, as
+// the Agents tab looks it up of those an agent marked (repository-agents.ts
+// pulledCommits), kept by the commit.
+function commitPulls(deps: ManagedHttpDeps, repoId: number): CommitPulls {
+  const store = commitPullStore<{ pulled: boolean }>(deps.db, deps.config.debug)
+  const keyOf = (sha: string) => `${repoId}\0${sha}`
+  return {
+    async read(shas) {
+      const kept = await store.readMany(shas.map(keyOf))
+      return new Map(shas.flatMap(sha => {
+        const pulled = kept.get(keyOf(sha))?.value?.pulled
+        return typeof pulled === 'boolean' ? [[sha, pulled] as const] : []
+      }))
+    },
+    write: (sha, pulled) => store.write(keyOf(sha), { pulled }, Date.now()),
+  }
+}
+
+// How long repository `repoId`'s pull requests kept (keptPulls) are taken as
+// they are before those updated since are read: less than its readers keep
+// their answers (keptAnswer), that one read of them serves both.
+const PULLS_FRESH_MS = 30_000
+// Each repository's reads of its pull requests, by its id: one at a time,
+// each after those asked before it.
+const pullsReads = new Map<number, Promise<unknown>>()
+
+// Repository `repoId`'s pull requests, as kept (repository-pulls.ts
+// readPulls), read on where they were read PULLS_FRESH_MS ago or longer, or
+// not as far back as `need`: as far back as it, for the Pulse; for the
+// Agents tab (null), as far back as they were read, those updated since read
+// (a read not finished, on), else their first page. One read of a
+// repository's at a time, the rest after it, from what it kept.
+function keptPulls(deps: ManagedHttpDeps, repoId: number, fullName: string, token: string, need: number | null): Promise<PullList> {
+  const store = pullListStore<PullList>(deps.db, deps.config.debug)
+  const key = `${repoId}\0${PULLS_READ_VERSION}\0${AGENTS_READ_VERSION}\0${PULSE_YEAR_VERSION}`
+  const reading = (pullsReads.get(repoId) ?? Promise.resolve()).catch(() => {}).then(async () => {
+    const kept = await store.read(key)
+    const list = kept?.value ?? null
+    if (list && Date.now() - kept!.at < PULLS_FRESH_MS && (need === null || pullsReadTo(list) <= need)) return list
+    const since = pulseSince(52, Date.now())
+    const read = await readPulls(fullName, token, since, list, globalThis.fetch,
+      need ?? (list ? list.next ? since : list.reach ?? since : Date.now()))
+    await store.write(key, read, Date.now())
+    return read
+  })
+  pullsReads.set(repoId, reading)
+  void reading.catch(() => {}).finally(() => { if (pullsReads.get(repoId) === reading) pullsReads.delete(repoId) })
+  return reading
+}
+
+// What the Pulse keeps of repository `repoId`'s pull requests merged, as
+// it tells by them whose its year's commits are (pulseCommits, agentMerges).
+const pulseMergesKey = (repoId: number) => `${repoId}\0pulse-merges\0${PULSE_YEAR_VERSION}`
+
+// The pull requests merged as the Pulse last read them through (a read not
+// finished, none), for the Commits tab to tell those squashed by
+// (repository-squashes.ts KnownMerges): each's merge commit and its number
+// (none of one read before numbers were), every one merged from as far
+// back as the read reaches (the year's start, or as far as a period needed)
+// till the newest read among them; null where none is kept.
+async function pulseMerges(deps: ManagedHttpDeps, repoId: number): Promise<KnownMerges | null> {
+  const kept = (await repositoryReadStore(deps.db, deps.config.debug).read(pulseMergesKey(repoId)))?.value as AgentMerges | undefined
+  if (!kept || kept.next || kept.through === null || !Array.isArray(kept.pulls)) return null
+  return { numbers: new Map(kept.pulls.map(([sha, , number]) => [sha, number ?? null])), from: Math.max(kept.reach ?? Number.NEGATIVE_INFINITY, pulseSince(52, Date.now())),
+    to: kept.through }
+}
+
 // The pull requests squashed into repository `repoId`'s commits, kept by
 // each commit and the rules their commits' co-authors were read by
 // (COAUTHORS_READ), null for one none was (repository-squashes.ts
@@ -2106,7 +2179,7 @@ function historySquashes(deps: ManagedHttpDeps, repoId: number): Squashes {
       const kept = await store.readMany(shas.map(keyOf))
       return new Map(shas.flatMap(sha => {
         const entry = kept.get(keyOf(sha))
-        return entry && (entry.value === null || (Array.isArray(entry.value.commits) && entry.value.commits.every(commit => Array.isArray(commit.coauthors))))
+        return entry && (entry.value === null || (Array.isArray(entry.value.commits) && entry.value.commits.every(commit => Array.isArray(commit.named))))
           ? [[sha, entry.value] as const] : []
       }))
     },
@@ -2213,9 +2286,10 @@ async function handleRepositoryIssues(res: ServerResponse, deps: ManagedHttpDeps
 
 // GET /api/admin/repositories/agents: what the code viewer's Agents tab
 // reconstructs agents' sessions from (repository-agents.ts): a page
-// of pull requests on from the cursor `pulls` names, and of the default
-// branch's commits on from `commits`, the first page, naming neither, with
-// issues and branches. They are the whole repository's (readWholeRepository).
+// of pull requests below the one the cursor `pulls` names, of those kept for
+// the Pulse too (keptPulls), and of the default branch's commits on from
+// `commits`, the first page, naming neither, with issues and branches. They
+// are the whole repository's (readWholeRepository).
 async function handleRepositoryAgents(res: ServerResponse, deps: ManagedHttpDeps, cookie: string | undefined, query: URLSearchParams): Promise<void> {
   const repoId = Number(query.get('repoId'))
   const cursors = { pulls: query.get('pulls'), commits: query.get('commits') }
@@ -2223,7 +2297,8 @@ async function handleRepositoryAgents(res: ServerResponse, deps: ManagedHttpDeps
     || (cursors.commits !== null && !COMMITS_CURSOR.test(cursors.commits))) {
     sendJson(res, 400, { error: 'bad-request' }); return
   }
-  await readWholeRepository(res, deps, cookie, repoId, (fullName, token) => repositoryAgents(fullName, token, cursors),
+  await readWholeRepository(res, deps, cookie, repoId, (fullName, token) => repositoryAgents(fullName, token, cursors, historyPages(deps, repoId, true, true),
+    commitPulls(deps, repoId), () => keptPulls(deps, repoId, fullName, token, null)),
     `agents\0${AGENTS_READ_VERSION}\0${cursors.pulls ?? ''}\0${cursors.commits ?? ''}`)
 }
 
@@ -2656,9 +2731,10 @@ async function handleRepositoryPulse(res: ServerResponse, deps: ManagedHttpDeps,
 // its commits as the Commits tab keeps them, with their lines
 // (yearCommits): kept by the head it was read from (repositoryReadStore),
 // that head kept as an issue's page is; its commits merged from an agent's
-// pull request told by those merged (agentMerges), kept as read, the rest
-// read on by the next answer as the year's commits are, and read again as
-// an answer kept is, a label changing whose a pull request is without
+// pull request told by those merged (agentMerges), of the pull requests
+// kept for the Agents tab too (keptPulls), what they tell kept as read, the
+// rest read on by the next answer as the year's commits are, and read again
+// as an answer kept is, a label changing whose a pull request is without
 // moving the head, the year counted again where what they bring changed.
 // Read no further back than the period asked (`weeks`) needs, its weeks and
 // as many before them, the rest read only as a longer one asks, its count
@@ -2718,25 +2794,26 @@ async function pulseCommits(deps: ManagedHttpDeps, repoId: number, fullName: str
   const store = repositoryReadStore(deps.db, deps.config.debug)
   const key = `${repoId}\0pulse-year\0${PULSE_YEAR_VERSION}\0${head.value.sha}\0${first}`
   const lastKey = `${repoId}\0pulse-year-last`
-  const mergesKey = `${repoId}\0pulse-merges\0${PULSE_YEAR_VERSION}`
+  const mergesKey = pulseMergesKey(repoId)
   const last = async () => {
     const kept = (await store.read(lastKey))?.value as { first: number; stats: PulseStats } | undefined
     return kept && first - kept.first <= PULSE_LAST_MS ? kept.stats : null
   }
   const sha = head.value.sha
   // What tells whose the year's commits are, kept as read: the agents' pull
-  // requests merged (agentMerges), and the commits that release the
+  // requests merged (agentMerges), as the pull requests kept tell, read as
+  // far back as `whole` (keptPulls), and the commits that release the
   // repository (`releases`, release-commits.ts releaseCommits).
   type Attribution = AgentMerges & { releases?: string[] }
   const keptAttribution = async () => ((await store.read(mergesKey))?.value as Attribution | undefined) ?? null
-  const attribution = async (kept: Attribution | null, whole: number): Promise<Attribution> => {
-    const [merges, releases] = await Promise.all([agentMerges(fullName, token, first, kept, globalThis.fetch, whole),
+  const attribution = async (whole: number): Promise<Attribution> => {
+    const [pulls, releases] = await Promise.all([keptPulls(deps, repoId, fullName, token, whole),
       releaseCommits(fullName, token, first, historyChanges(deps, repoId))])
-    return { ...merges, releases }
+    return { ...agentMerges(pulls, first), releases }
   }
-  // Since when every pull request merged is read (agentMerges `reach`, or
-  // a read not finished's `to`): all of them, -∞; none yet, ∞.
-  const readTo = (read: Attribution) => read.next ? read.next.to ?? Number.POSITIVE_INFINITY : read.reach ?? Number.NEGATIVE_INFINITY
+  // Since when every pull request merged is read (repository-pulls.ts
+  // pullsReadTo): all of them, -∞; none yet, ∞.
+  const readTo = (read: Attribution) => pullsReadTo(read)
   // The year counted from its `commits` as `read` tells whose they are and
   // which were pushed directly, the branch followed through those it was
   // read on past (`passed`, yearCommits) too, whole from when both the
@@ -2761,7 +2838,7 @@ async function pulseCommits(deps: ManagedHttpDeps, repoId: number, fullName: str
     const whole = stats.from ?? first
     await keptAnswer(deps, mergesKey, async () => {
       const kept = await keptAttribution()
-      const read = await attribution(kept, whole)
+      const read = await attribution(whole)
       const same = kept && sameMerges(read, kept) && read.releases!.join() === (kept.releases ?? []).join() && readTo(kept) <= readTo(read)
       if (readTo(read) >= whole || same) return read
       const year = await yearCommits(fullName, token, sha, first, historyPages(deps, repoId, true), globalThis.fetch, whole)
@@ -2779,7 +2856,7 @@ async function pulseCommits(deps: ManagedHttpDeps, repoId: number, fullName: str
   }
   if (!held || !('tooMany' in held)) {
     const [year, read] = await Promise.all([yearCommits(fullName, token, sha, first, historyPages(deps, repoId, true), globalThis.fetch, need),
-      keptAttribution().then(kept => attribution(kept, need))])
+      attribution(need)])
     await store.write(mergesKey, read, Date.now())
     if (year.complete) {
       if (readTo(read) >= need) return { stats: await last(), reading: null, pulls: read.next?.read ?? 0, pending: false, kept: false }
@@ -3714,11 +3791,12 @@ async function handleQueryReports(req: IncomingMessage, res: ServerResponse, dep
 
 // The server selects the complete published workspace for classification.
 // Privileged hidden-report previews are independently scoped and bounded.
-// Its reports' file trees are asked for apart, by its graph or Files alone.
+// Its reports' file trees are asked for apart, by its graph or Files alone,
+// and its board's reports without their findings' detail (`?lite=1`).
 async function handleTeamReports(res: ServerResponse, deps: ManagedHttpDeps, session: ManagedSession, teamId: string, reportId: string | null,
-  part: 'reports' | 'trees' = 'reports'): Promise<void> {
+  part: TeamResponsePart = 'reports'): Promise<void> {
   const snapshot = await teamSnapshot(deps.db, session.id, teamId, reportId)
-  const body = await (part === 'trees' ? loadTeamTreesResponse : loadTeamReportsResponse)(deps.db, deps.reportStore, snapshot)
+  const body = await loadTeamReportsResponse(deps.db, deps.reportStore, snapshot, part)
   await recheckTeam(deps.db, session.id, snapshot)
   res.writeHead(200, { 'content-type': 'application/json', 'cache-control': 'no-store', 'x-content-type-options': 'nosniff' })
   writeResponse(res, body)
@@ -5092,7 +5170,7 @@ export function createManagedRequestHandler(deps: ManagedHttpDeps): Handler {
     const teamReports = /^\/api\/teams\/([^/]+)\/(reports|trees)$/u.exec(path)
     if (teamReports) {
       if (method !== 'GET') { send405(res, 'GET'); return }
-      await handleTeamReports(res, deps, workspaceSession!.session, teamReports[1]!, url.searchParams.get('reportId'), teamReports[2] as 'reports' | 'trees'); return
+      await handleTeamReports(res, deps, workspaceSession!.session, teamReports[1]!, url.searchParams.get('reportId'), teamResponsePart(teamReports[2]!, url)); return
     }
     if (path === REPORT_QUERY_PATH) {
       if (method !== 'POST') { send405(res, 'POST'); return }

package/server-managed/npm-kept.ts

@@ -161,6 +161,14 @@ export const historyPageStore = <T>(db: UpstreamCacheStore, debug = false): Kept
 // The pull requests squashed into a repository's commits, by the commit
 // (repository-squashes.ts squashedPulls): kept as its history's pages are.
 export const historySquashStore = <T>(db: UpstreamCacheStore, debug = false): KeptStore<T> => keptStore<T>(db, 'repository/squashes', HISTORY_PAGE_KEPT_MS, debug)
+// Whether a pull request brought a commit, by its repository and commit, as
+// the Agents tab looks it up of those an agent marked (repository-agents.ts
+// pulledCommits), kept as the history is.
+export const commitPullStore = <T>(db: UpstreamCacheStore, debug = false): KeptStore<T> => keptStore<T>(db, 'repository/commit-pulls', HISTORY_PAGE_KEPT_MS, debug)
+// A repository's pull requests, the most recently updated first, as the
+// Pulse and the Agents tab read them (repository-pulls.ts readPulls), by its
+// id: kept as its history's pages are, those updated since read on.
+export const pullListStore = <T>(db: UpstreamCacheStore, debug = false): KeptStore<T> => keptStore<T>(db, 'repository/pulls', HISTORY_PAGE_KEPT_MS, debug)
 // The files each of a repository's commits changes (repository-changes.ts
 // repositoryChanges), by the commit, which never change: kept as its
 // history's pages are.

package/server-managed/public-workspace.ts

@@ -7,7 +7,7 @@ import type { OpenedBlob } from './blob-store.ts'
 import type { WorkspaceShareSnapshot } from './workspace-shares.ts'
 import { sendJson, sendLines, writeResponse } from './http-response.ts'
 import { hashToken } from './crypto.ts'
-import { TeamReportsError, loadTeamAnnotations, loadTeamReportsResponse, loadTeamTreesResponse, teamReportVisibility } from './team-reports.ts'
+import { TeamReportsError, loadTeamAnnotations, loadTeamReportsResponse, teamReportVisibility, teamResponsePart } from './team-reports.ts'
 import { triageWireEntry } from './triage-response.ts'
 import { MAX_PACKAGE_INVENTORY_BYTES } from './bundle-cache.ts'
 import { backfillBundleSummaries, bundleSummaries } from './bundle-catalog.ts'
@@ -84,7 +84,7 @@ export async function handlePublicWorkspace(req: IncomingMessage, res: ServerRes
       await send(await loadTeamAnnotations(deps.db, deps.reportStore, snapshot, url.searchParams.get('reportId'))); return
     }
     if (teamRoute[2] === 'shared') { await sendTeam(); return }
-    const body = await (teamRoute[2] === 'trees' ? loadTeamTreesResponse : loadTeamReportsResponse)(deps.db, deps.reportStore, snapshot)
+    const body = await loadTeamReportsResponse(deps.db, deps.reportStore, snapshot, teamResponsePart(teamRoute[2]!, url))
     await recheck()
     res.writeHead(200, { 'content-type': 'application/json', 'cache-control': 'private, no-store', 'referrer-policy': 'no-referrer' })
     writeResponse(res, body); return