Repository navigation
MP-78: /open: скоуп и чат организации для межпроектной работы - #81
Merged
Merged
Conversation
…e organization keys
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Goal
Adds a fourth session scope, the organization, so that one chat can coordinate work across all registered projects of one organization. The example is TB-FF, whose plan touches several of its repositories.
/open tb-ff(MCPopen_session, CLImypi open) opens, focuses or resumes the organization session.org-<slug>in workspace<slug>, and the session starts in the engine root.turn_endandwaitinggo to whoever sent the message that started the turn.globalandallare refused with clear errors.Plan: https://draft.yokeloop.com/artifacts/mp-78-open-e8cilgx9
Changes
scripts/pipeline/mp-task scope.sessionsgainsscope_type = 'org'and anorg_idforeign key withON DELETE RESTRICT. Theused_project_codetrigger is recreated.org removerefuses while sessions or events reference the organization.PARENT_MESSAGEregex stay identical.orgrole, global and organization memory, and a compact## Projectssection.integrations/promptswere condensed so the 9500-character Claude warmup budget still holds.mp-task scopegains theorgandcodesfields. Step 0 of the/plan,/do,/flow,/shipand/boardskills anddocs/PIPELINE.mdcover the organization case.docs/{MCP,M1-CLI,ARCHITECTURE,HTTP-API,PIPELINE,TESTING,MANUAL-ACCEPTANCE}.mdand the.yoke/context.mdglossary.src/mcp/**and the CLI usage text.Checks
mise exec -- pnpm build: exit 0.mise exec -- pnpm verifyon d6780e6: exit 0. Fast 178/0, boundary 65/0, wall 49.4 s, memory peak 526 MiB.Review
Three rounds by a fresh reviewer each time; final verdict: approved.
task_updatedescription and the schema-12 statement in HTTP-API.md were out of date. Advice covered the parent rules,/boardtexts, a test for the regex copy and the CLI usage text. All were fixed.send_messagedescription was out of date. Fixed. The warmup texts were also condensed after a stand measurement: the real global warmup would otherwise have gone about 30 characters over the 9500 budget. The global golden grew by only 98 characters over the base (8416 → 8514).Open advice (not applied):
integrations/prompts/howto.md:11says "unless that chat made it". A reader can take "that chat" to mean only the project chat. The code and every other text mean the recipient. A clearer wording would be "unless the recipient made it".Live acceptance (disposable stand)
tb-ff: TBA and TBD with clones, HSK without a clone;two: OT;empty.nope,all,globalandempty.no clone, no project memory.turn_endarrives live in the organization chat and not in TBA./opencompletions show only the organization's projects and tasks./boardshows the organization's boards and/plan OT-1is refused.org remove tb-ffis refused.--max-chars 9500: global 8793, org 8187, project 8026, task 7898./openstart without isolation flags and load the user's personal extensions.Not part of this PR
mypi db initfor schema 13 on the installed engine is a separately authorized step after merge.🤖 Generated with Claude Code