mirror of
https://github.com/alkimake/paperclip.git
synced 2026-06-15 18:30:39 +09:00
## Thinking Path > - Paperclip orchestrates AI agents for zero-human companies > - The board depends on issue, inbox, cost, and company-skill surfaces to stay accurate and fast while agents are actively working > - The PAP-1497 follow-up branch exposed a few rough edges in those surfaces: stale active-run state on completed issues, missing creator filters, oversized issue payload scans, and placeholder issue-route parsing > - Those gaps make the control plane harder to trust because operators can see misleading run state, miss the right subset of work, or pay extra query/render cost on large issue records > - This pull request tightens those follow-ups across server and UI code, and adds regression coverage for the affected paths > - The benefit is a more reliable issue workflow, safer high-volume cost aggregation, and clearer board/operator navigation ## What Changed - Added the `v2026.415.0` release changelog entry. - Fixed stale issue-run presentation after completion and reused the shared issue-path parser so literal route placeholders no longer become issue links. - Added creator filters to the Issues page and Inbox, including persisted filter-state normalization and regression coverage. - Bounded issue detail/list project-mention scans and trimmed large issue-list payload fields to keep issue reads lighter. - Hardened company-skill list projection and cost/finance aggregation so large markdown blobs and large summed values do not leak into list responses or overflow 32-bit casts. - Added targeted server/UI regression tests for company skills, costs/finance, issue mention scanning, creator filters, inbox normalization, and issue reference parsing. ## Verification - `pnpm exec vitest run server/src/__tests__/company-skills-service.test.ts server/src/__tests__/costs-service.test.ts server/src/__tests__/issues-goal-context-routes.test.ts server/src/__tests__/issues-service.test.ts ui/src/lib/inbox.test.ts ui/src/lib/issue-filters.test.ts ui/src/lib/issue-reference.test.ts` - `gh pr checks 3779` Current pass set on the PR head: `policy`, `verify`, `e2e`, `security/snyk (cryppadotta)`, `Greptile Review` ## Risks - Creator filter options are derived from the currently loaded issue/agent data, so very sparse result sets may not surface every historical creator until they appear in the active dataset. - Cost/finance aggregate casts now use `double precision`; that removes the current overflow risk, but future schema changes should keep large-value aggregation behavior under review. - Issue detail mention scanning now skips comment-body scans on the detail route, so any consumer that relied on comment-only project mentions there would need to fetch them separately. ## Model Used - OpenAI Codex, GPT-5-based coding agent with terminal tool use and local code execution in the Paperclip workspace. Exact internal model ID/context-window exposure is not surfaced in this session. ## Checklist - [x] I have included a thinking path that traces from project context to this change - [x] I have specified the model used (with version and capability details) - [x] I have run tests locally and they pass - [x] I have added or updated tests where applicable - [ ] If this change affects the UI, I have included before/after screenshots - [x] I have updated relevant documentation to reflect my changes - [x] I have considered and documented any risks above - [x] I will address all Greptile and reviewer comments before requesting merge --------- Co-authored-by: Paperclip <noreply@paperclip.ing>
245 lines
7.4 KiB
TypeScript
245 lines
7.4 KiB
TypeScript
import express from "express";
|
|
import request from "supertest";
|
|
import { beforeEach, describe, expect, it, vi } from "vitest";
|
|
|
|
const mockIssueService = vi.hoisted(() => ({
|
|
getById: vi.fn(),
|
|
getAncestors: vi.fn(),
|
|
getRelationSummaries: vi.fn(),
|
|
findMentionedProjectIds: vi.fn(),
|
|
getCommentCursor: vi.fn(),
|
|
getComment: vi.fn(),
|
|
listAttachments: vi.fn(),
|
|
}));
|
|
|
|
const mockProjectService = vi.hoisted(() => ({
|
|
getById: vi.fn(),
|
|
listByIds: vi.fn(),
|
|
}));
|
|
|
|
const mockGoalService = vi.hoisted(() => ({
|
|
getById: vi.fn(),
|
|
getDefaultCompanyGoal: vi.fn(),
|
|
}));
|
|
|
|
vi.mock("../services/index.js", () => ({
|
|
accessService: () => ({
|
|
canUser: vi.fn(),
|
|
hasPermission: vi.fn(),
|
|
}),
|
|
agentService: () => ({
|
|
getById: vi.fn(),
|
|
}),
|
|
documentService: () => ({
|
|
getIssueDocumentPayload: vi.fn(async () => ({})),
|
|
}),
|
|
executionWorkspaceService: () => ({
|
|
getById: vi.fn(),
|
|
}),
|
|
feedbackService: () => ({
|
|
listIssueVotesForUser: vi.fn(async () => []),
|
|
saveIssueVote: vi.fn(async () => ({ vote: null, consentEnabledNow: false, sharingEnabled: false })),
|
|
}),
|
|
goalService: () => mockGoalService,
|
|
heartbeatService: () => ({
|
|
wakeup: vi.fn(async () => undefined),
|
|
reportRunActivity: vi.fn(async () => undefined),
|
|
}),
|
|
instanceSettingsService: () => ({
|
|
get: vi.fn(async () => ({
|
|
id: "instance-settings-1",
|
|
general: {
|
|
censorUsernameInLogs: false,
|
|
feedbackDataSharingPreference: "prompt",
|
|
},
|
|
})),
|
|
listCompanyIds: vi.fn(async () => ["company-1"]),
|
|
}),
|
|
issueApprovalService: () => ({}),
|
|
issueService: () => mockIssueService,
|
|
logActivity: vi.fn(async () => undefined),
|
|
projectService: () => mockProjectService,
|
|
routineService: () => ({
|
|
syncRunStatusForIssue: vi.fn(async () => undefined),
|
|
}),
|
|
workProductService: () => ({
|
|
listForIssue: vi.fn(async () => []),
|
|
}),
|
|
}));
|
|
|
|
async function createApp() {
|
|
const [{ issueRoutes }, { errorHandler }] = await Promise.all([
|
|
vi.importActual<typeof import("../routes/issues.js")>("../routes/issues.js"),
|
|
vi.importActual<typeof import("../middleware/index.js")>("../middleware/index.js"),
|
|
]);
|
|
const app = express();
|
|
app.use(express.json());
|
|
app.use((req, _res, next) => {
|
|
(req as any).actor = {
|
|
type: "board",
|
|
userId: "local-board",
|
|
companyIds: ["company-1"],
|
|
source: "local_implicit",
|
|
isInstanceAdmin: false,
|
|
};
|
|
next();
|
|
});
|
|
app.use("/api", issueRoutes({} as any, {} as any));
|
|
app.use(errorHandler);
|
|
return app;
|
|
}
|
|
|
|
const legacyProjectLinkedIssue = {
|
|
id: "11111111-1111-4111-8111-111111111111",
|
|
companyId: "company-1",
|
|
identifier: "PAP-581",
|
|
title: "Legacy onboarding task",
|
|
description: "Seed the first CEO task",
|
|
status: "todo",
|
|
priority: "medium",
|
|
projectId: "22222222-2222-4222-8222-222222222222",
|
|
goalId: null,
|
|
parentId: null,
|
|
assigneeAgentId: "33333333-3333-4333-8333-333333333333",
|
|
assigneeUserId: null,
|
|
updatedAt: new Date("2026-03-24T12:00:00Z"),
|
|
executionWorkspaceId: null,
|
|
labels: [],
|
|
labelIds: [],
|
|
};
|
|
|
|
const projectGoal = {
|
|
id: "44444444-4444-4444-8444-444444444444",
|
|
companyId: "company-1",
|
|
title: "Launch the company",
|
|
description: null,
|
|
level: "company",
|
|
status: "active",
|
|
parentId: null,
|
|
ownerAgentId: null,
|
|
createdAt: new Date("2026-03-20T00:00:00Z"),
|
|
updatedAt: new Date("2026-03-20T00:00:00Z"),
|
|
};
|
|
|
|
describe("issue goal context routes", () => {
|
|
beforeEach(() => {
|
|
vi.resetModules();
|
|
vi.doUnmock("../routes/issues.js");
|
|
vi.doUnmock("../routes/authz.js");
|
|
vi.doUnmock("../middleware/index.js");
|
|
vi.resetAllMocks();
|
|
mockIssueService.getById.mockResolvedValue(legacyProjectLinkedIssue);
|
|
mockIssueService.getAncestors.mockResolvedValue([]);
|
|
mockIssueService.getRelationSummaries.mockResolvedValue({ blockedBy: [], blocks: [] });
|
|
mockIssueService.findMentionedProjectIds.mockResolvedValue([]);
|
|
mockIssueService.getCommentCursor.mockResolvedValue({
|
|
totalComments: 0,
|
|
latestCommentId: null,
|
|
latestCommentAt: null,
|
|
});
|
|
mockIssueService.getComment.mockResolvedValue(null);
|
|
mockIssueService.listAttachments.mockResolvedValue([]);
|
|
mockProjectService.getById.mockResolvedValue({
|
|
id: legacyProjectLinkedIssue.projectId,
|
|
companyId: "company-1",
|
|
urlKey: "onboarding",
|
|
goalId: projectGoal.id,
|
|
goalIds: [projectGoal.id],
|
|
goals: [{ id: projectGoal.id, title: projectGoal.title }],
|
|
name: "Onboarding",
|
|
description: null,
|
|
status: "in_progress",
|
|
leadAgentId: null,
|
|
targetDate: null,
|
|
color: null,
|
|
pauseReason: null,
|
|
pausedAt: null,
|
|
executionWorkspacePolicy: null,
|
|
codebase: {
|
|
workspaceId: null,
|
|
repoUrl: null,
|
|
repoRef: null,
|
|
defaultRef: null,
|
|
repoName: null,
|
|
localFolder: null,
|
|
managedFolder: "/tmp/company-1/project-1",
|
|
effectiveLocalFolder: "/tmp/company-1/project-1",
|
|
origin: "managed_checkout",
|
|
},
|
|
workspaces: [],
|
|
primaryWorkspace: null,
|
|
archivedAt: null,
|
|
createdAt: new Date("2026-03-20T00:00:00Z"),
|
|
updatedAt: new Date("2026-03-20T00:00:00Z"),
|
|
});
|
|
mockProjectService.listByIds.mockResolvedValue([]);
|
|
mockGoalService.getById.mockImplementation(async (id: string) =>
|
|
id === projectGoal.id ? projectGoal : null,
|
|
);
|
|
mockGoalService.getDefaultCompanyGoal.mockResolvedValue(null);
|
|
});
|
|
|
|
it("surfaces the project goal from GET /issues/:id when the issue has no direct goal", async () => {
|
|
const res = await request(await createApp()).get("/api/issues/11111111-1111-4111-8111-111111111111");
|
|
|
|
expect(res.status).toBe(200);
|
|
expect(res.body.goalId).toBe(projectGoal.id);
|
|
expect(res.body.goal).toEqual(
|
|
expect.objectContaining({
|
|
id: projectGoal.id,
|
|
title: projectGoal.title,
|
|
}),
|
|
);
|
|
expect(mockIssueService.findMentionedProjectIds).toHaveBeenCalledWith(
|
|
"11111111-1111-4111-8111-111111111111",
|
|
{ includeCommentBodies: false },
|
|
);
|
|
expect(mockGoalService.getDefaultCompanyGoal).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it("surfaces the project goal from GET /issues/:id/heartbeat-context", async () => {
|
|
const res = await request(await createApp()).get(
|
|
"/api/issues/11111111-1111-4111-8111-111111111111/heartbeat-context",
|
|
);
|
|
|
|
expect(res.status).toBe(200);
|
|
expect(res.body.issue.goalId).toBe(projectGoal.id);
|
|
expect(res.body.goal).toEqual(
|
|
expect.objectContaining({
|
|
id: projectGoal.id,
|
|
title: projectGoal.title,
|
|
}),
|
|
);
|
|
expect(mockGoalService.getDefaultCompanyGoal).not.toHaveBeenCalled();
|
|
expect(res.body.attachments).toEqual([]);
|
|
});
|
|
|
|
it("surfaces blocker summaries on GET /issues/:id/heartbeat-context", async () => {
|
|
mockIssueService.getRelationSummaries.mockResolvedValue({
|
|
blockedBy: [
|
|
{
|
|
id: "55555555-5555-4555-8555-555555555555",
|
|
identifier: "PAP-580",
|
|
title: "Finish wakeup plumbing",
|
|
status: "done",
|
|
priority: "medium",
|
|
assigneeAgentId: null,
|
|
assigneeUserId: null,
|
|
},
|
|
],
|
|
blocks: [],
|
|
});
|
|
|
|
const res = await request(await createApp()).get(
|
|
"/api/issues/11111111-1111-4111-8111-111111111111/heartbeat-context",
|
|
);
|
|
|
|
expect(res.status).toBe(200);
|
|
expect(res.body.issue.blockedBy).toEqual([
|
|
expect.objectContaining({
|
|
id: "55555555-5555-4555-8555-555555555555",
|
|
identifier: "PAP-580",
|
|
}),
|
|
]);
|
|
});
|
|
});
|