Skip to content

feat(planning): Planning Kanban board UI + graph toggle - #445

Merged
github-actions[bot] merged 1 commit into
feature/planning-kanban-datafrom
feature/planning-kanban-ui
Jul 3, 2026
Merged

feat(planning): Planning Kanban board UI + graph toggle#445
github-actions[bot] merged 1 commit into
feature/planning-kanban-datafrom
feature/planning-kanban-ui

Conversation

@tomymaritano

Copy link
Copy Markdown
Collaborator

Summary

The UI half of the Planning Kanban feature. Stacked on #444 (the data layer) — this PR's base is feature/planning-kanban-data and will auto-retarget to develop once #444 merges. Review #444 first.

Adds the "Planning" sidebar item and the Kanban board it opens. Columns are the 5 Linear-style stages; dragging a card persists via notes.setBoardStageDB metadata only, the note's markdown is never rewritten ("markdown is sacred").

What's included

Navigation

  • NavigationState gains a planning kind + goToPlanning action + selectIsPlanningContext
  • Sidebar Planning quick-filter item (KanbanSquare), active-state aware
  • App.tsx renders <PlanningBoard> in the editor pane when navigation.kind === 'planning'

Board (components/planning/)

  • PlanningBoard — groups Planning-notebook notes into Backlog/Todo/In Progress/In Review/In Staging; Board ↔ Graph toggle
  • PlanningColumn — droppable column (native HTML5 DnD, custom application/x-dripnex-note MIME — the app's DnD convention, no new dep)
  • PlanningCard — draggable card: title, excerpt, tags, task progress via countMarkdownTasks; click opens the note
  • Graph mode reuses GraphView via a new optional filterNotebookId prop that restricts nodes/edges to the Planning notebook

Hooks: useNoteMutations.setBoardStage; BoardStage re-exported from preload.

Testing

  • pnpm typecheck (all desktop projects) — green
  • pnpm test — green; pnpm lint — green
  • pnpm --filter @dripnex/desktop build — bundles successfully
  • Manual smoke test pending on pnpm dev (Planning item → board → drag persists → markdown unchanged → graph toggle)

How it connects to the graph

No new graph infra — the existing GraphView / useGraphData / window.dripnex.links are reused; filterNotebookId scopes them to the board's notes (nodes already carry notebookId).

🤖 Generated with Claude Code

Adds the "Planning" sidebar item and the Kanban board it opens, built on
the boardStage data layer. Dragging a card between columns persists via
notes.setBoardStage (DB metadata only — the note's markdown is untouched).

Navigation:
- NavigationState gains a 'planning' kind + goToPlanning action + selector
- Sidebar "Planning" quick-filter item (KanbanSquare), active-state aware
- App.tsx renders <PlanningBoard> in the editor pane for kind==='planning'

Board:
- PlanningBoard: groups Planning-notebook notes into the 5 Linear-style
  columns (Backlog/Todo/In Progress/In Review/In Staging); Board|Graph toggle
- PlanningColumn: droppable column (native HTML5 DnD, custom MIME)
- PlanningCard: draggable card with title, excerpt, tags, and task progress
  via countMarkdownTasks; click opens the note
- Graph mode reuses GraphView with a new filterNotebookId prop that restricts
  nodes/edges to the Planning notebook

Hooks: useNoteMutations.setBoardStage; BoardStage re-exported from preload.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)
  • main
  • develop

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 8b8ef735-a347-42ca-be17-dd9305c8a17a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/planning-kanban-ui

Comment @coderabbitai help to get the list of available commands.

@github-actions
github-actions Bot merged commit 219da96 into feature/planning-kanban-data Jul 3, 2026
4 of 5 checks passed
tomymaritano added a commit that referenced this pull request Jul 3, 2026
## Problem

The Auto-merge workflow enabled auto-merge on **every** owner PR on
`opened`/`synchronize`/`reopened`/`ready_for_review`. Consequences:

- PRs to `develop` had auto-merge enabled automatically, so they merged
the moment branch protection was satisfied — **before CodeRabbit review
was resolved** (the exact CodeRabbit-first violation we keep hitting).
- **Stacked PRs targeting an unprotected feature branch merged
instantly** (no protection to hold them). This is what collapsed the
stacked UI PR (#445) into its base branch before review.

## Fix

Owner auto-merge is now **opt-in via the `automerge` label**:
- The `automerge-owner` job only runs when the PR carries the
`automerge` label, and the workflow now also reacts to the `labeled`
event.
- Add the label when a PR is genuinely ready to merge; leave it off (or
remove it) to hold — no more surprise merges.
- Dependabot auto-merge is unchanged.

The `automerge` label has been created in the repo.

## Testing
Workflow-only change (YAML). Verified the `if:` uses
`contains(github.event.pull_request.labels.*.name, 'automerge')` and the
`labeled` trigger is present so applying the label enables auto-merge.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
* Updated automated merge behavior to require both a matching PR author
and an automation label before merging.
* Expanded event handling so labeled pull requests are evaluated for
auto-merge eligibility.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant