Skip to content

docs(specs): multi-orchestrator handoff — learnings & carry-forward#149

Open
Weegy wants to merge 1 commit into
mainfrom
docs/multi-orchestrator-learnings
Open

docs(specs): multi-orchestrator handoff — learnings & carry-forward#149
Weegy wants to merge 1 commit into
mainfrom
docs/multi-orchestrator-learnings

Conversation

@Weegy
Copy link
Copy Markdown
Contributor

@Weegy Weegy commented May 27, 2026

Summary

Post-Phase-B retrospective handoff. Captures the 10 top learnings from
shipping PRs #146, #148 (omadia) and omadia-byte5-plugins#3 live —
plus 8 carry-forward items and operational recipes.

Top 10 covered

  1. Per-Agent orchestrators built before kernel DomainTools exist → empty tool surface
  2. agent_plugins.config persisted but not yet runtime-applied
  3. Microsoft Teams conversation.id semantics are non-obvious
  4. Node ESM module cache breaks plugin hot-reload of sub-imports
  5. replaceAgentPlugins re-upserts stale rows if the UI echoes them back
  6. agent.updated_at doesn't bump on plugin/binding writes
  7. validateSnapshot rejects the entire snapshot when one plugin is "not installed"
  8. Auto-merge via GraphQL is rate-limit-fragile; REST merge is the workhorse
  9. usePathname not window.location.pathname + popstate
  10. Header z-index needs an explicit stacking context

Carry-forward (linked to learnings)

  1. Per-(Agent × plugin) PluginContext refactor — unblocks L2
  2. Plugin module-cache fix — L4
  3. Conversation-observer persistence
  4. Azure Bot Service display rename (manual)
  5. Telegram webhook handler via channelResolver@1
  6. Snapshot validation per-row isolation — L7
  7. Operator-side per-plugin config runtime semantics
  8. T043–T046 (structured logging, Notion docs sync, two-Agent smoke, quickstart e2e)

Test plan

Documentation-only — no code changes.

🤖 Generated with Claude Code

Post-Phase-B retrospective. Covers the 10 top learnings from shipping
PRs #146, #148 (omadia) and byte5-plugins #3 live:
DomainTool late-binding, runtime-vs-persisted agent_plugins.config,
Teams conversation-id mechanics, Node ESM module-cache pitfall on
plugin hot-reload, replaceAgentPlugins re-upsert, updated_at not
bumping on child writes, snapshot validation isolation, GraphQL
rate-limit vs REST merge, usePathname vs popstate, header z-index.
Plus 8 carry-forward items and operational recipes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant