Skip to content

refactor(task-board): ♻️ Align task summary row - #152

Merged
jorben merged 2 commits into
masterfrom
fix/task-board-layout
Apr 29, 2026
Merged

refactor(task-board): ♻️ Align task summary row#152
jorben merged 2 commits into
masterfrom
fix/task-board-layout

Conversation

@jorben

@jorben jorben commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Align the task board summary into a single horizontal row.
  • Keep Current: ... left-aligned and the completed count right-aligned.
  • Prevent the progress count from wrapping while preserving active-task truncation.

Test Plan

  • npm run test:unit -- task-board-card.test.tsx
  • npm run typecheck

🤖 Generated with TiyCode

…k layout

Align task count and active task description side by side using flexbox, improving vertical space usage and visual consistency. The active task description now truncates properly and is placed next to the summary instead of below it.
@github-actions

github-actions Bot commented Apr 28, 2026

Copy link
Copy Markdown

AI Code Review Summary

PR: #152 (refactor(task-board): ♻️ Align task summary row)
Preferred language: English

Overall Assessment

No blocking issue was detected in the reviewed diff; keep focused regression testing before merge.

Major Findings by Severity

No major issues identified from the reviewed diff.

Actionable Suggestions

  • Add a nullish fallback for activeTask.description to avoid showing 'undefined'.

Potential Risks

  • If task objects can have empty descriptions, the UI may display 'Current:' with no meaningful text.

Test Suggestions

  • Add a unit test for task-board-card rendering when activeTask.description is missing.

File-Level Coverage Notes

  • src/modules/workbench-shell/ui/task-board-card.tsx: Safe layout refactor that keeps existing behavior. Minor edge case with missing description text should be hardened.

Inline Downgraded Items (processed but not inline)

  • None

Coverage Status

  • Target files: 1
  • Covered files: 1
  • Uncovered files: 0
  • No-patch/binary covered as file-level: 0
  • Findings with unknown confidence (N/A): 0

Uncovered list:

  • None

No-patch covered list:

  • None

Runtime/Budget

  • Rounds used: 1/4
  • Planned batches: 1
  • Executed batches: 1
  • Sub-agent runs: 1
  • Planner calls: 1
  • Reviewer calls: 1
  • Model calls: 2/64
  • Structured-output summary-only degradation: NO

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated PR review completed.

  • Findings kept: 0
  • Findings with unknown confidence: 0
  • Inline comments attempted: 1
  • Target files: 1
  • Covered files: 1
  • Uncovered files: 0
    See the summary comment for detailed analysis and coverage details.

@@ -170,20 +170,20 @@ export const TaskBoardCard = ({
</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review completed for this PR diff. No concrete inline issue was selected after aggregation.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated PR review completed.

  • Findings kept: 0
  • Findings with unknown confidence: 0
  • Inline comments attempted: 1
  • Target files: 1
  • Covered files: 1
  • Uncovered files: 0
    See the summary comment for detailed analysis and coverage details.

@@ -170,20 +170,20 @@ export const TaskBoardCard = ({
</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review completed for this PR diff. No concrete inline issue was selected after aggregation.

@jorben
jorben merged commit 8ac8edb into master Apr 29, 2026
4 checks passed
@jorben
jorben deleted the fix/task-board-layout branch April 29, 2026 02:10
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