Skip to content

Drop pi-intercom and move oracle/reviewer to Opus 4.8 - #97

Merged
boga merged 2 commits into
masterfrom
fix-plan-interaction
Jun 17, 2026
Merged

Drop pi-intercom and move oracle/reviewer to Opus 4.8#97
boga merged 2 commits into
masterfrom
fix-plan-interaction

Conversation

@boga

@boga boga commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Why

Subagents hung indefinitely whenever one called contact_supervisor during a
chain or parallel run. That tool is a blocking intercom ask: the child freezes
waiting for a reply the parent can never send (the parent is itself blocked
inside the subagent() call, or the ask lands in an exited coordinator
session). There is no timeout, so the hang is permanent.

Rather than restrict each agent's toolset, this removes the source: dropping
the pi-intercom extension removes the contact_supervisor/intercom tools
entirely, so no agent can issue a blocking ask.

Separately, oracle and reviewer are promoted to a stronger reasoning model.

What

  • Drop npm:pi-intercom from pi_agent_settings_packages (group_vars/all.yml).
    Removes contact_supervisor globally — fixes the hang at the root.
  • oracle + revieweranthropic/claude-opus-4-8, thinking high
    on both hosts:
    • work: was google/gemini-3.1-pro-preview.
    • home: was openai-codex/gpt-5.5; also added anthropic/claude-opus-4-8
      to enabledModels so the override resolves there.

Notes

  • User-scoped config; no project files affected.
  • Takes effect after applying the playbook
    (ansible-playbook site.yml --limit <host> --tags configs). The settings
    merge is recursive/additive — re-running drops pi-intercom from the
    generated settings.json packages list.
  • The plan's optional tools: allowlists were intentionally skipped; dropping
    the extension makes them redundant.

Summary by CodeRabbit

  • Chores
    • Removed deprecated package dependency to streamline configuration
    • Updated underlying AI models powering core agents for enhanced performance

Remove npm:pi-intercom from pi_agent_settings_packages. Dropping the
extension removes the blocking contact_supervisor tool, fixing the
indefinite hang in chain/parallel subagent runs at the root.

Switch oracle and reviewer subagents to anthropic/claude-opus-4-8 with
high thinking on both work and home hosts. Add opus-4-8 to home's
enabledModels so the override resolves there.
@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 65a19f91-34ac-470f-b6f4-8df2501bf2e9

📥 Commits

Reviewing files that changed from the base of the PR and between 90776c4 and 5b8b0fa.

📒 Files selected for processing (2)
  • group_vars/all.yml
  • host_vars/work/vars.yml

📝 Walkthrough

Walkthrough

Removes npm:pi-intercom from the global pi_agent_settings_packages list and switches the oracle and reviewer subagent model values in host_vars/work/vars.yml from google/gemini-3.1-pro-preview to anthropic/claude-opus-4-8.

Changes

Config updates: package removal and model changes

Layer / File(s) Summary
Remove npm:pi-intercom from global packages
group_vars/all.yml
Removes the npm:pi-intercom entry from pi_agent_settings_packages, making the list flow directly from npm:context-mode to npm:pi-mcp-adapter.
Switch oracle and reviewer subagent models to Claude Opus 4
host_vars/work/vars.yml
Updates oracle and reviewer subagent model values from google/gemini-3.1-pro-preview to anthropic/claude-opus-4-8 in pi_agent_subagent_overrides.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • boga/dotfiles#90: Directly inverse change — modifies the same oracle and reviewer subagent model fields in host_vars/work/vars.yml, switching to google/gemini-3.1-pro-preview instead of anthropic/claude-opus-4-8.

Poem

🐇 A package removed, a model switched anew,
From Gemini's glow to Claude's deeper hue,
The oracle speaks in Anthropic's voice,
The reviewer too — a deliberate choice.
Hop, hop, configs tidy and true! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-plan-interaction

Comment @coderabbitai help to get the list of available commands and usage tips.

@boga boga changed the title chore(pi-agent): drop pi-intercom, move oracle/reviewer to opus-4-8 Drop pi-intercom and move oracle/reviewer to Opus 4.8 Jun 17, 2026
Revert the home host changes from f3e034d. oracle and reviewer return to
openai-codex/gpt-5.5 (xhigh), and anthropic/claude-opus-4-8 is removed
from home's enabledModels. The work host change is unaffected.
@boga
boga marked this pull request as ready for review June 17, 2026 10:52
@boga
boga merged commit cdf299e into master Jun 17, 2026
1 check was pending
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