Skip to content

skills: make comsolcompile/comsolbatch a first-class execution path#30

Merged
jiweiqi merged 1 commit into
mainfrom
batch-first-class-path
May 14, 2026
Merged

skills: make comsolcompile/comsolbatch a first-class execution path#30
jiweiqi merged 1 commit into
mainfrom
batch-first-class-path

Conversation

@jiweiqi
Copy link
Copy Markdown
Contributor

@jiweiqi jiweiqi commented May 14, 2026

What

The bundled COMSOL skill framed the comsolcompile/comsolbatch one-shot Java path as a second-class fallback and a "not yet implemented" feature. Both framings were inaccurate — the batch executables are COMSOL's own and an agent can invoke them directly today. This reframes batch as a first-class COMSOL execution path with honest, merit-based selection criteria, and aligns the skill with the plugin-skill authoring guideline (present sim-cli as one valid path among several).

Changes

  • Decision table — batch row rewritten with merit-based criteria (settled deterministic recipes, reproducibility/isolation, fan-out, minimal lifecycle); sim-runtime row rebalanced to point at batch where it fits.
  • New "Choosing between live session and batch" section — the explore-live → solidify → graduate-to-batch composition arc.
  • Headless section — renamed; "not yet implemented" re-scoped to the sim-cli driver wrapper, not the underlying capability.
  • java_batch_patterns.md — opening reframed as a first-class execution mode.
  • Intro, routing rule, Required-protocol step 1 — reframed to present sim-cli as one path among several rather than the default.
  • Removed SKILL.zh-CN.md — an unreferenced companion (loaded by nothing, only this plugin had one) that had already drifted from SKILL.md. Skills are English-only.

Documentation-only; no driver code touched.

🤖 Generated with Claude Code

The COMSOL skill framed the comsolcompile/comsolbatch one-shot Java path
as a second-class fallback and a "not yet implemented" feature. Neither
held up: the batch executables are COMSOL's own and an agent can call
them directly today.

- Decision table: replace the batch row's fallback framing with
  merit-based criteria (settled deterministic recipes, reproducibility,
  fan-out, minimal lifecycle); rebalance the sim-runtime row to point at
  batch where it fits.
- Add a "Choosing between live session and batch" section: the
  explore-live -> solidify -> graduate-to-batch arc.
- Rename the headless section; scope "not yet implemented" to the
  sim-cli driver wrapper, not the capability itself.
- java_batch_patterns.md: present it as a first-class execution mode.
- Reframe the intro, routing rule, and Required-protocol step 1 to
  present sim-cli as one path among several, per the skill-authoring
  guideline.
- Remove the unreferenced SKILL.zh-CN.md companion; skills are
  English-only and the file had already drifted from SKILL.md.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jiweiqi jiweiqi marked this pull request as ready for review May 14, 2026 14:09
@jiweiqi jiweiqi merged commit 8ddaaee into main May 14, 2026
1 check passed
@jiweiqi jiweiqi deleted the batch-first-class-path branch May 14, 2026 14:09
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