Skip to content

feat(pm): expand MCP, routines, work items, inbox, and skills - #844

Open
Neonforge98 wants to merge 43 commits into
developfrom
feat/pm-expansion-batch
Open

feat(pm): expand MCP, routines, work items, inbox, and skills#844
Neonforge98 wants to merge 43 commits into
developfrom
feat/pm-expansion-batch

Conversation

@Neonforge98

@Neonforge98 Neonforge98 commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Problem

The PM parity branch contained useful MCP, Routine, Work Item, Views, Inbox, Skills, Quick Actions, queueing, and provider fixes, but it had also accumulated a Work Item-specific remote continuation bridge. That bridge implied a durable dispatcher from Cloud comments to another device and duplicated Session continuation responsibilities that belong to the existing Team/My Session model.

The remaining PM work also needed one reviewable description of its security and compatibility boundaries: MCP policy/secrets, Work Item routing and revision safety, Routine scheduling/cancellation, typed properties and saved views, Inbox lifecycle, Skills provenance, provider-backed suggestions, and bounded durable chat queues.

Solution

  • Preserve the PM parity work: MCP policy and secret redaction; portable Routine execution; List/Table/Kanban/Gantt/Calendar and saved views; typed properties/statuses; Quick Actions; Inbox lifecycle; Skills provenance/consent; revision CAS; context-overflow recovery; and bounded durable queues.
  • Keep provider execution on the normal local Session/provider paths, including the resolved MCP set and provider account/model configuration.
  • Remove the Work Item remote continuation RPC, conversation_turn_bridge, local continuation registry/runner, and Discussion bypass. Work Item comments now stay on the established comment/Work Run path instead of defining a second conversation system.
  • Preserve all PM schema and compatibility behavior already introduced by the parity batch; the cleanup adds no replacement Cloud runner, checkpoint, E2EE payload, or execution-host concept.
  • Merge current develop so feat(conversations): unify native session continuation #939 can stack on one stable PM head.

Potential risks

  • This remains a deliberately large PM parity checkpoint. The touched areas share Work Item execution and persistence boundaries, but the review surface is larger than the repository default.
  • Schema changes are additive and old serde payloads default safely. Rollback should revert the eventual merge commit; inert additive tables/columns can remain without destructive cleanup.
  • MCP read/modify/write serialization is process-local, so independent application processes editing the same config remain last-writer-wins.
  • Agent reply persistence and deferred-escalation cancellation are separate transactions; a crash between them can leave a fallback pending until a matching receipt or terminal event.
  • Queue persistence is bounded and crash-safe, but a provider accepting a send immediately before a local crash may still need explicit dispatch recovery.
  • The removed remote continuation bridge is intentionally not replaced here. Generic transcript continuation is isolated in stacked PR feat(conversations): unify native session continuation #939 and executes only on the sender's active local device.

Verification

  • cargo test -p project_management — 677 passed.
  • cargo test -p orgtrack-pm-cli — 17 passed.
  • Focused Work Run Rust suites — 25 passed.
  • Focused changed frontend suites — 63 passed.
  • pnpm exec tsc --noEmit — passed.
  • Targeted ESLint — passed.
  • Scoped Clippy — passed.
  • git diff --check and changed-file secret/personal-path audit — passed.
  • Confirmed the removed bridge/runner identifiers have no remaining references in the changed production scope.

Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
… edit

Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
@Neonforge98
Neonforge98 marked this pull request as draft August 20, 2026 03:15
Pre-commit hook ran. Total eslint: 5, total circular: 0
@Neonforge98 Neonforge98 changed the title feat(pm): MCP injection, discussion routing v2, custom statuses, saved views, table view, quick actions, inbox lifecycle feat(pm): complete multica parity across MCP, routines, work items, inbox, and skills Aug 20, 2026
@Neonforge98
Neonforge98 marked this pull request as ready for review August 20, 2026 05:15
@Neonforge98
Neonforge98 marked this pull request as draft August 20, 2026 05:15
…batch

Pre-commit hook ran. Total eslint: 5, total circular: 0
@Neonforge98
Neonforge98 marked this pull request as ready for review August 20, 2026 05:31
@Neonforge98
Neonforge98 marked this pull request as draft August 20, 2026 07:27
Pre-commit hook ran. Total eslint: 5, total circular: 0
…batch

Pre-commit hook ran. Total eslint: 5, total circular: 0
@Neonforge98
Neonforge98 marked this pull request as ready for review August 20, 2026 07:42
@Neonforge98
Neonforge98 marked this pull request as draft August 20, 2026 08:34
Pre-commit hook ran. Total eslint: 5, total circular: 0
@Neonforge98
Neonforge98 marked this pull request as ready for review August 20, 2026 08:48
Pre-commit hook ran. Total eslint: 5, total circular: 0
@Neonforge98
Neonforge98 force-pushed the feat/pm-expansion-batch branch from 437b322 to a7e9043 Compare August 20, 2026 21:48
@Neonforge98 Neonforge98 changed the title feat(pm): complete multica parity across MCP, routines, work items, inbox, and skills feat(pm): expand MCP, routines, work items, inbox, and skills Aug 20, 2026
…batch

Pre-commit hook ran. Total eslint: 5, total circular: 0
…cesses

Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
Pre-commit hook ran. Total eslint: 5, total circular: 0
mcp-injection-live.spec.mjs proves the .orgii/mcp-servers.json probe server
reaches a spawned CLI session end-to-end: the rendered transcript must show
the MCP tool block and the assistant must echo the probe marker. Provider
rows skip as blocked when no usable account exists. The marker is asserted
against the settled chat body because the driver can sample the reply while
it still renders a loading placeholder.

The routine-wizard scenario adds an extra activation through the rendered
controls, asserts the persisted list mirrors the trigger in slot 0 (cron
triggers map to schedule activations), round-trips the extra cron in edit
mode, and clears the explicit list when the row is removed.

Pre-commit hook ran. Total eslint: 5, total circular: 0
…batch

# Conflicts:
#	src/modules/ProjectManager/WorkItems/components/WorkItemContent/CustomPropertiesSection.tsx
#	src/modules/ProjectManager/WorkItems/components/WorkItemContent/index.tsx

Pre-commit hook ran. Total eslint: 5, total circular: 0
The multi-select bar gains Set status, Set priority and Set assignee
dialogs next to Set property, and the board can group by project or by
a custom property.

Pre-commit hook ran. Total eslint: 1, total circular: 0
@Harry19081 Harry19081 added enhancement New feature or request project-management Projects, work items, routines, GitHub work, or team inbox agent Agent runtime, behavior, memory, providers, or orchestration labels Aug 23, 2026
Project-mode briefs for native and external-CLI sessions now embed the
org's active custom status catalog plus status and mention discipline
lines, and org2-pm work transition/list accept active custom keys.

Pre-commit hook ran. Total eslint: 1, total circular: 0
Discussion reply turns that resume a session no longer wait on the
coding run's path lock, retry() returns the open retry episode instead
of stacking children, and a manual launch blocked by the checkout lock
stays queued for the dispatcher and reports that instead of failing.

Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
…batch

# Conflicts:
#	src-tauri/crates/project-management/src/projects/io/routines.rs
#	src-tauri/src/agent_sessions/cli/persistence/mod.rs
#	src/api/http/project/client.ts
#	src/api/http/project/client/workItems.test.ts
#	src/modules/MainApp/TeamInbox/TeamInboxView.tsx
#	src/modules/MainApp/TeamInbox/domain/index.ts
#	src/modules/MainApp/TeamInbox/domain/selectors.ts
#	src/modules/ProjectManager/WorkItems/components/WorkItemContent/index.tsx

Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 1, total circular: 0
Pre-commit hook ran. Total eslint: 0, total circular: 0
Pre-commit hook ran. Total eslint: 0, total circular: 0
Keep the existing Team Session one-shot behavior while deleting the remote-root sender-device handoff, hidden runner continuation registry, and its Work Run transport plumbing. Preserve independent PM queueing and retry convergence fixes.
…20260826

# Conflicts:
#	src-tauri/crates/agent-core/src/core/tools/impls/coding/exec/subprocess.rs
#	src-tauri/crates/orgtrack-pm-cli/src/commands.rs
#	src-tauri/crates/project-management/src/work_run_service/mod.rs
#	src-tauri/crates/project-management/src/work_service/mod.rs
#	src/modules/MainApp/TeamInbox/components/AssignedWorkItemDetail.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent Agent runtime, behavior, memory, providers, or orchestration enhancement New feature or request project-management Projects, work items, routines, GitHub work, or team inbox

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants