Skip to content

fix(dashboard): review summaries from project cards#74

Merged
Vann-Dev merged 2 commits into
mainfrom
kaito/project-card-summary-review
May 17, 2026
Merged

fix(dashboard): review summaries from project cards#74
Vann-Dev merged 2 commits into
mainfrom
kaito/project-card-summary-review

Conversation

@Vann-Dev
Copy link
Copy Markdown
Owner

Summary

  • add a project-board server action to mark done summaries reviewed by the configured main/Natsuki review reader
  • add a compact card-level reviewed button for unread done summaries without changing drag/drop or context menu handling
  • show stored summary freshness on expanded card previews

Tests

  • corepack pnpm --filter @agentbridge/web lint
  • corepack pnpm --filter @agentbridge/web typecheck
  • corepack pnpm --filter @agentbridge/web build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agent-bridge-web Ready Ready Preview, Comment May 17, 2026 5:33am

@Vann-Dev
Copy link
Copy Markdown
Owner Author

Updated PR #74 for Ume QA feedback.\n\nChanges:\n- Project card done-summary unread lookup now resolves the same review reader via findReviewReader() used by markProjectTaskSummaryReadAction instead of hard-coding AgentId main.\n- Added shared read-marker where helper and focused coverage for non-main review-reader agent id.\n\nChecks:\n- corepack pnpm test:api ✅\n- corepack pnpm lint ✅\n- DATABASE_URL=postgresql://placeholder:placeholder@localhost:5432/placeholder corepack pnpm typecheck:web ✅\n- DATABASE_URL=postgresql://placeholder:placeholder@localhost:5432/placeholder corepack pnpm build:web ✅ (Next warns middleware convention is deprecated)\n\nCommit: 3005cea

@Vann-Dev Vann-Dev merged commit 1ed7917 into main May 17, 2026
3 checks passed
@Vann-Dev Vann-Dev deleted the kaito/project-card-summary-review branch May 17, 2026 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant