Skip to content

wave3(interview): import skill to 3B + fix dead engine link#5

Merged
brandonwie merged 2 commits intomainfrom
wave3/interview-skill-import
Apr 26, 2026
Merged

wave3(interview): import skill to 3B + fix dead engine link#5
brandonwie merged 2 commits intomainfrom
wave3/interview-skill-import

Conversation

@brandonwie
Copy link
Copy Markdown
Owner

@brandonwie brandonwie commented Apr 26, 2026

Summary

Wave 3 sync for the interview skill. Two atomic commits:

  1. docs(skills/interview) — replace dead plugins/interview-codex/ pointer (directory never existed in forge) with the real Python engine at plugins/3b/engine/. Adds concrete trigger conditions surfaced by plugin review (numeric scoring 0–1, 40/30/30 weighting, file-locked persisted InterviewState, shared agents/ SSoT).
  2. chore(manifest) — record 11 new SOURCE-MANIFEST entries for the Wave 3 interview-skill sync (1 SKILL + 3 references + 7 perspectives). 3B is canonical (matches Wave 2 direction). Top metadata bumped: synced_at2026-04-26, three_b_head_at_syncc547ebfe.

Context

Companion to 3B branch feat/embed-interview-skill which imports the skill into 3B as canonical. Drift script (scripts/check-3b-drift.sh) shows 29/29 entries pass.

Forge agents/ layout (plugin-root) differs from 3B's nested layout (.claude/skills/interview/agents/) — drift script compares content not paths so divergence is intentional.

Test plan

  • bash scripts/check-3b-drift.sh → 29/29 pass
  • grep -rn 'interview-codex' plugins/3b/skills/interview/ → no stale references
  • forge engine path verified: plugins/3b/engine/{README.md,src,tests,pyproject.toml} exist

Summary by CodeRabbit

  • Chores

    • Advanced internal synchronization metadata to Wave 3 with expanded configuration mappings for system integration
  • Documentation

    • Updated implementation guidance clarifying ambiguity scoring methodology on a 0-1 scale with specified weighting, state persistence mechanics, and shared resource references for programmatic integrators

brandonwie and others added 2 commits April 26, 2026 23:26
…ine link

Stale link to plugins/interview-codex/ — directory does not exist.
Real Python engine lives at plugins/3b/engine/. Updated pointer +
added concrete trigger conditions (numeric scoring 0–1 with 40/30/30
weighting, file-locked persisted InterviewState, shared agents/ SSoT).
Surfaced by 3b-forge plugin review during Wave 3 (interview skill
import to 3B).

Co-authored-by: claude <noreply@anthropic.com>
Adds 11 manifest entries covering interview skill SSoT:
- 1 SKILL.md
- 3 references (claude-code/codex/gemini-tools.md)
- 7 perspective agents (socratic-interviewer, researcher, simplifier,
  architect, breadth-keeper, seed-closer, ontologist)

3B is canonical for Wave 3 (matches Wave 2 direction). Forge agents/
layout differs from 3B's nested layout — drift script compares
content not paths so the divergence is intentional.

Top metadata bumped: synced_at 2026-04-24 → 2026-04-26,
three_b_head_at_sync f9c6d19c → c547ebfe (3B HEAD at Wave 3 sync).

Co-authored-by: claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 26, 2026 14:27
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 26, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6d12431d-f158-4c4e-8e3d-f2f90aaded18

📥 Commits

Reviewing files that changed from the base of the PR and between f56e066 and 36d6b2c.

📒 Files selected for processing (2)
  • plugins/3b/SOURCE-MANIFEST.yaml
  • plugins/3b/skills/interview/SKILL.md

📝 Walkthrough

Walkthrough

The 3B plugin's source manifest is advanced to Wave 3 with updated synchronization metadata and 11 new interview skill mappings, alongside documentation clarifications for the Python engine variant configuration.

Changes

Cohort / File(s) Summary
Wave 3 Manifest & Sync Metadata
plugins/3b/SOURCE-MANIFEST.yaml
Updated sync metadata (synced_at, three_b_head_at_sync) to Wave 3 SHA c547ebfe57..., added 11 new Wave 3 interview entry mappings (4 skills including 3 reference files, 7 agent entries) pointing from plugins/3b/ to .claude/skills/interview/ paths with updated source_sha and empty scrub lists.
Documentation Updates
plugins/3b/skills/interview/SKILL.md
Revised Python-engine variant guidance with 0–1 ambiguity scale weighting (40/30/30), clarified state persistence via file-locked InterviewState, updated engine reference path to ../../engine/, and added notes on shared agents/ SSoT and opt-in availability for programmatic integrators.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly Related PRs

Poem

🐰 Wave 3 hops in with fresh mappings bright,
Metadata synced to a sparkling new light,
Ambiguity scales and state files align,
The interview engine now runs so fine! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch wave3/interview-skill-import

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@brandonwie brandonwie merged commit b24d4c2 into main Apr 26, 2026
2 of 3 checks passed
@brandonwie brandonwie deleted the wave3/interview-skill-import branch April 26, 2026 14:27
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Syncs Wave 3 content for the interview skill into the forge plugins/3b area by fixing a stale documentation link and registering the imported skill + related reference/agent files in the 3B source manifest for drift checking.

Changes:

  • Update interview skill docs to replace a dead plugins/interview-codex/ pointer with the actual Python engine location at plugins/3b/engine/, and document scoring/state details.
  • Bump manifest top metadata (synced_at, three_b_head_at_sync) for the Wave 3 sync.
  • Add 11 new Wave 3 manifest entries (skill, references, and agent perspective files) pointing at the canonical 3B sources.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
plugins/3b/skills/interview/SKILL.md Fixes the engine link and clarifies the Python-engine variant details (scoring/state/SSoT).
plugins/3b/SOURCE-MANIFEST.yaml Updates Wave 3 sync metadata and records the new interview-skill related source mappings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants