Repository navigation
Route Knowledge links to their owning apps - #78
Conversation
|
🚀 personal-todo build ready for testing Artifact: Download from workflow run Playground ZIP: Launch in Playground: Open WordPress Playground Paste the ZIP URL into a WordPress Playground blueprint to load this PR build. |
|
🚀 personal-stuff build ready for testing Artifact: Download from workflow run Playground ZIP: Launch in Playground: Open WordPress Playground Paste the ZIP URL into a WordPress Playground blueprint to load this PR build. |
|
🚀 personal-ai-chat build ready for testing Artifact: Download from workflow run Playground ZIP: Launch in Playground: Open WordPress Playground Paste the ZIP URL into a WordPress Playground blueprint to load this PR build. |
|
🚀 personal-readwise-sync build ready for testing Artifact: Download from workflow run Playground ZIP: Launch in Playground: Open WordPress Playground Paste the ZIP URL into a WordPress Playground blueprint to load this PR build. |
|
🚀 personal-notes build ready for testing Artifact: Download from workflow run Playground ZIP: Launch in Playground: Open WordPress Playground Paste the ZIP URL into a WordPress Playground blueprint to load this PR build. |
|
🚀 personal-evernote-sync build ready for testing Artifact: Download from workflow run Playground ZIP: Launch in Playground: Open WordPress Playground Paste the ZIP URL into a WordPress Playground blueprint to load this PR build. |
There was a problem hiding this comment.
🟡 Changes recommended
AI Chat deep links outside the first 20 conversations silently open the wrong conversation.
1 open finding
What changed in this PR
Routes Knowledge records to their owning split-plugin interfaces.
Changes:
- Adds package-owned permalink, shortlink, and legacy-link routing.
- Adds URL-backed TODO and AI Chat selection.
- Documents and tests routing contracts and ability normalization.
| File | Description |
|---|---|
tests/unit/SplitPackageKnowledgeTest.php |
Tests package link ownership and abilities output. |
tests/unit/PersonalStuffTest.php |
Tests Stuff item links. |
packages/personal-todo/TECHNICAL.md |
Documents TODO link routing. |
packages/personal-todo/src/index.js |
Synchronizes task editing with browser history. |
packages/personal-todo/includes/class-personal-todo-plugin.php |
Routes task links to TODO. |
packages/personal-stuff/TECHNICAL.md |
Documents Stuff link routing. |
packages/personal-stuff/includes/class-personal-stuff-plugin.php |
Routes item links to Stuff. |
packages/personal-notes/TECHNICAL.md |
Documents Notes link routing. |
packages/personal-notes/includes/class-personal-notes-plugin.php |
Routes note links to Gutenberg. |
packages/personal-ai-chat/TECHNICAL.md |
Documents conversation routing. |
packages/personal-ai-chat/src/index.js |
Adds URL-backed conversation selection. |
packages/personal-ai-chat/includes/class-personal-ai-chat-plugin.php |
Routes conversations and normalizes abilities. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| const selected = | ||
| response.find( ( item ) => item.id === conversationId ) || | ||
| response[ 0 ] || | ||
| null; |
Summary
?p=<id>requests for its canonical Knowledge identityValidation
npm run lint:js -- packages/personal-todo/src/index.js packages/personal-ai-chat/src/index.jsnpm run test:unit:backend(174 tests, 1070 assertions, 10 skipped)npm run test:integration:backend(suite contains one skipped test)npm run buildnpm run package:pluginsnpm run verify:split-packages?p=requests for Notes, TODO, Stuff, and AI Chat opened the owning interface and selected the requested record