Skip to content
This repository was archived by the owner on May 29, 2026. It is now read-only.

chore(openspec): leaf-migration specs for p4-collaboration (ADR-022)#269

Merged
rubenvdlinde merged 1 commit into
developmentfrom
chore/leaf-migration-specs
May 25, 2026
Merged

chore(openspec): leaf-migration specs for p4-collaboration (ADR-022)#269
rubenvdlinde merged 1 commit into
developmentfrom
chore/leaf-migration-specs

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Summary

Five OpenSpec migration changes (specs only — no code) that move decidesk's archived p4-collaboration in-app service stack onto OpenRegister integration leaves, per hydra ADR-022 (apps consume OR abstractions; leaf-first + catalogue) and ADR-019 (integration registry). Decidesk is already a registry consumer via MeetingIntegrations.vue (xWiki leaf live), so each change reuses that proven tab/widget shell.

Change In-app service retired Leaf
migrate-comments-to-talk-leaf CommentService talk
migrate-action-items-to-deck-leaf TaskService / DelegationService deck
migrate-workspaces-to-collectives-leaf WorkspaceService collectives
migrate-engagement-analytics-to-analytics-leaf EngagementService / ActionItemAnalyticsService / VotingBehaviourService analytics
migrate-email-links-to-email-leaf EmailLinkService email

Each change has proposal.md, design.md, a specs/<cap>/spec.md delta (ADDED + REMOVED requirements), and tasks.md. All five pass openspec validate --strict.

Key decisions

  • ActionItems VTODO-vs-deck reconciliation: CalDAV VTODO ActionItems (ADR-002) remain the source of truth for action-item content; the deck leaf is the board UI / projection. The p4 Task schema is retired (not a third store); delegation/reclaim map onto VTODO assignee changes + OR audit events.
  • Analytics split: chart/dashboard rendering and generic aggregations move to the analytics leaf (or x-openregister-aggregations, ADR-031); governance-specific derived metrics the leaf can't compute (engagement-score formula, quorum/weight-aware voting behaviour) stay in-app and feed the leaf as values.
  • All migrations archive legacy objects (OR archival workflow), never hard-delete — audit trail preserved.

ADR-022 exceptions (kept in-app — NOT migrated), documented in each design.md

  • Statutory votingVotingService / QuorumService / LiveDecisionService (secret ballots, quorum, proxy/weighted votes), incl. MailReplyHandler vote-by-email casting. Polls leaf is for informal straw polls only.
  • ORI / Popolo publication — ADR-001 / ADR-003.

Out of scope

Specs only. Implementation runs through Hydra later. Do not merge until reviewed.

…ADR-022)

Add five OpenSpec migration changes moving decidesk's in-app
p4-collaboration stack onto OpenRegister integration leaves (ADR-019 /
ADR-022, leaf-first + catalogue):

- migrate-comments-to-talk-leaf        (CommentService -> talk)
- migrate-action-items-to-deck-leaf    (TaskService/DelegationService -> deck;
                                        VTODO stays source of truth per ADR-002)
- migrate-workspaces-to-collectives-leaf (WorkspaceService -> collectives)
- migrate-engagement-analytics-to-analytics-leaf (Engagement/*Analytics/
                                        VotingBehaviour dashboards -> analytics;
                                        domain metric calc kept in-app)
- migrate-email-links-to-email-leaf    (EmailLinkService -> email; vote-by-email
                                        MailReplyHandler stays statutory)

Documented ADR-022 exceptions in each design.md: statutory voting
(VotingService/QuorumService/LiveDecisionService) and ORI/Popolo
publication (ADR-001/003) stay in-app. Specs only; apply via Hydra.
@rubenvdlinde rubenvdlinde requested a review from Rem-Dam as a code owner May 25, 2026 23:20
@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/decidesk @ 536e8c8

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer
npm ✅ 383/383
PHPUnit ⏭️
Newman ⏭️
Playwright ⏭️

Quality workflow — 2026-05-25 23:22 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde rubenvdlinde merged commit e198f8d into development May 25, 2026
13 of 22 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant