Skip to content

feat(examples): add engineering workspace - #454

Open
christso wants to merge 7 commits into
mainfrom
feat/engineering-plugin
Open

feat(examples): add engineering workspace#454
christso wants to merge 7 commits into
mainfrom
feat/engineering-plugin

Conversation

@christso

@christso christso commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add the project-scoped engineering workspace with the complete Compound Engineering plugin
  • pin Compound to compound-engineering-v3.23.3 instead of maintaining a near-complete skill allowlist
  • preserve Matt Pocock's exact nine-skill upstream functional closure and Hermes llm-wiki
  • keep ce-worktree available; Workmux remains the preferred future orchestration path once its runtime prerequisites land

Configuration

version: 2

repositories: []

plugins:
  # Intentionally enable the complete Compound Engineering plugin. The release
  # pin keeps every included skill and asset reproducible across updates.
  - source: https://github.com/EveryInc/compound-engineering-plugin
    pin: compound-engineering-v3.23.3

  # Preserve Matt's upstream composition by installing the selected capabilities
  # with every helper skill they invoke. The explicit closure prevents unrelated
  # additions from auto-enabling while keeping each selected workflow complete.
  - source: https://github.com/mattpocock/skills
    skills:
      - engineering/wayfinder
      - engineering/grill-with-docs
      - engineering/improve-codebase-architecture
      - engineering/domain-modeling
      - engineering/research
      - engineering/prototype
      - engineering/codebase-design
      - engineering/setup-matt-pocock-skills
      - productivity/grilling

  - source: https://github.com/NousResearch/hermes-agent
    skills:
      - research/llm-wiki

clients:
  - universal

Pin evidence

  • git ls-remote --tags resolves compound-engineering-v3.23.3 to 871996c4e35512e3d33cdc9c2532395f7bf10831
  • upstream .claude-plugin/plugin.json reports name compound-engineering, version 3.23.3
  • isolated AllAgents sync-state records resolvedRef: compound-engineering-v3.23.3, resolvedSha: 871996c4e35512e3d33cdc9c2532395f7bf10831, and pinnedRef: compound-engineering-v3.23.3

Resolved skills

Exactly 43 skills are enabled: 33 Compound + 9 Matt + 1 Hermes.

Compound (33):

  • ce-babysit-pr
  • ce-brainstorm
  • ce-code-review
  • ce-commit
  • ce-commit-push-pr
  • ce-compound
  • ce-compound-refresh
  • ce-debug
  • ce-doc-review
  • ce-dogfood
  • ce-explain
  • ce-handoff
  • ce-ideate
  • ce-optimize
  • ce-plan
  • ce-polish
  • ce-pov
  • ce-product-pulse
  • ce-promote
  • ce-proof
  • ce-prototype
  • ce-resolve-pr-feedback
  • ce-retune
  • ce-riffrec-feedback-analysis
  • ce-setup
  • ce-simplify-code
  • ce-strategy
  • ce-sweep
  • ce-test-browser
  • ce-test-xcode
  • ce-work
  • ce-worktree
  • lfg

Matt (9): codebase-design, domain-modeling, grill-with-docs, grilling, improve-codebase-architecture, prototype, research, setup-matt-pocock-skills, wayfinder.

Hermes (1): llm-wiki.

Isolated install verification

From a standalone copy containing only examples/workspaces/engineering, with disposable HOME, TMPDIR, and XDG config/cache/data directories:

  • actual project-scoped update --json: 43 copied, 0 failed, 0 skipped
  • plugin results: Compound 33/0 failures, Matt 9/0 failures, Hermes 1/0 failures
  • project-scoped skill list --json: exactly 43 enabled skills with the 33/9/1 split above
  • Compound's resolved set exactly matches all 33 top-level directories at the pinned release
  • recursive SHA-256 comparison: 33 Compound skill directories, 392 files, 0 missing, 0 extra, 0 changed
  • pinned cache contains .claude-plugin/CHANGELOG.md, marketplace.json, and plugin.json; metadata reports version 3.23.3
  • no .claude-plugin or .agents/plugins metadata was copied into the universal client surface, as expected
  • upstream root .agents/skills/ce-skill-work did not become client-visible; no nested .agents/skills entry exists under any installed skill
  • all three configured sources are remote HTTPS URLs, so the standalone copy required no source files from the AllAgents checkout

Focused tests

  • bun test tests/unit/utils/workspace-parser.test.ts tests/unit/utils/source-parser.test.ts tests/unit/models/workspace-config.test.ts tests/unit/core/plugin.test.ts tests/unit/core/skill-resolution.test.ts — 54 pass, 0 fail
  • git diff --check — clean
  • project-wide formatter, lint, and full test suite intentionally not run

Local handoff commit: 137b80f (fix(workspace): enable complete Compound plugin). The coordinator will merge/push; this session did not push directly.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 23, 2026

Copy link
Copy Markdown

Deploying allagents with  Cloudflare Pages  Cloudflare Pages

Latest commit: cb4c579
Status: ✅  Deploy successful!
Preview URL: https://0f5f1863.allagents.pages.dev
Branch Preview URL: https://feat-engineering-plugin.allagents.pages.dev

View logs

@christso

Copy link
Copy Markdown
Contributor Author

Expanded the Compound allowlist based on actual workflow usage. Added exact upstream names ce-ideate, ce-brainstorm, ce-plan, ce-simplify-code, ce-debug, ce-pov, and ce-work, plus the shipping dependency closure ce-commit, ce-commit-push-pr, ce-babysit-pr, and ce-resolve-pr-feedback.

Reverification: focused parser/resolution tests 19/19; isolated project-scoped update copied exactly 17 enabled skills with 0 failures (13 Compound, 3 Matt, 1 Hermes).

@christso

Copy link
Copy Markdown
Contributor Author

Replaced the repository-owned Matt adaptations with the exact upstream dependency closure.

Decision: the selected workflows are composed upstream skills. Vendoring narrowed research, prototype, tracker, report, domain, and parallel-design behavior while creating a maintenance fork. Shared Matt triage remains excluded because its policy is not project-discovered; Compound continues to own plan/work/review/debug.

Verification:

  • current Matt main is still audited commit 5b15a47f2d7150f545fbcacbfe381787fc0230dc; all nine qualified paths are unchanged
  • focused parser/source/skill-resolution tests: 42 pass, 0 fail
  • isolated standalone project update: exactly 23 copied, 0 failed (13 Compound, 9 Matt, 1 Hermes)
  • documented in-place example update: exactly 23 copied, 0 failed
  • all nine installed Matt directories match fetched upstream directories; sibling assets are present
  • no unresolved static named skill calls outside the closure; Wayfinder ## Notes skill names are dynamic issue-map data, and setup only conditionally detects optional triage

Local coordinator commit: e1b8ff7 (fix(workspace): use upstream Matt skill closure). Not pushed directly.

@christso

Copy link
Copy Markdown
Contributor Author

Compound dependency/handoff closure is complete locally in 2b35a07 (fix(workspace): enable reachable Compound handoffs). Per coordination instructions, the commit was not pushed.

Closure decision:

  • enabled exactly 24 Compound skills: ce-promote, ce-code-review, ce-ideate, ce-brainstorm, ce-plan, ce-simplify-code, ce-debug, ce-pov, ce-work, ce-commit, ce-commit-push-pr, ce-babysit-pr, ce-resolve-pr-feedback, ce-doc-review, ce-prototype, lfg, ce-test-browser, ce-proof, ce-explain, ce-compound, ce-compound-refresh, ce-handoff, ce-polish, ce-setup
  • retained Matt's 9-skill closure and Hermes llm-wiki
  • deliberately excluded ce-optimize (advisory mention only), ce-strategy (top-level intent boundary), and ce-worktree (honors only a pre-existing user request)
  • upstream Compound default branch is still audited commit 871996c4e35512e3d33cdc9c2532395f7bf10831; no required skill path/name changed

Verification:

  • focused parser/source/name/resolution tests: 51 pass, 0 fail
  • actual project-scoped source-CLI update from a standalone copy of the checked-in example with isolated HOME/XDG config/cache: 34 copied, 0 failed, 0 skipped
  • project skill list: exactly 34 enabled = 24 Compound + 9 Matt + 1 Hermes
  • scanned all 282 installed Compound SKILL.md/reference files; every hard, conditional, and user-visible explicit target resolves
  • documentation examples and project-dynamic calls were separated from static dependencies; ce-product-pulse/ce-sweep occur only in commented config examples, while project-defined shipping/review calls resolve from active project instructions
  • recursive hash comparison for the 11 newly enabled Compound skills: 123 files, 0 missing, 0 extra, 0 changed; sibling references, scripts, personas, and assets are intact

The PR description now contains the full closure rationale, exact resolved set, verification, and deliberate exclusions.

@christso

Copy link
Copy Markdown
Contributor Author

Full-plugin cutover is implemented and verified in local commit 137b80f (not pushed directly).

Evidence:

  • compound-engineering-v3.23.3 -> 871996c4e35512e3d33cdc9c2532395f7bf10831; plugin metadata version 3.23.3
  • isolated standalone project update: 43 copied, 0 failed, 0 skipped
  • exact enabled split: Compound 33 + Matt 9 + Hermes 1
  • all 392 files across the 33 Compound skill directories matched the pinned cache recursively; no missing/extra/changed assets
  • root .agents/skills/ce-skill-work and other nested .agents/skills content did not become client-visible
  • focused parser/source/config/plugin/skill-resolution tests: 54 pass, 0 fail

The complete plugin intentionally retains ce-worktree. Workmux can be added as the preferred orchestration path after runtime prerequisite support lands.

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