molecule-ai/molecule-dev-department
35
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat(defaults): add molecule-skill-five-axis-review plugin #6

Open
claude-ceo-assistant wants to merge 1 commits from feat/add-five-axis-review-plugin into main
pull from: feat/add-five-axis-review-plugin
merge into: molecule-ai:main
Conversation 0 Commits 1 Files Changed 1 +1
claude-ceo-assistant commented 2026-05-08 13:07:59 +00:00
Owner
Copy Link

Wires the new SOP Phase 4 self-review skill into dev-department.yaml defaults so every dev workspace (Dev Lead, Core Platform, Controlplane, App-Docs, Infra, SDK + floaters + Triage Operator) picks it up automatically.

What

   plugins:
     - molecule-skill-cron-learnings
     - molecule-skill-update-docs
+    - molecule-skill-five-axis-review  # SOP Phase 4 — structured five-axis self-review before PR

Why

Five-Axis self-review on the !external resolver work (internal#77, PRs #105+#106 of molecule-core) caught three Required-severity issues that the same author's unstructured 3-weakest pass on the same code missed:

Issue Caught by
Cache-validity gap Correctness axis
Token persistence in .git/config Security axis
Misleading function name Readability axis

Setting it as a default rather than per-workspace because every agent doing substantive change benefits from the structured pass — it's the default mode of operation, not an opt-in.

Plugin contract

Per org-template plugin resolution (PR #71, issue #68 of molecule-core) the defaults.plugins UNIONs with per-workspace plugins: lists. Workspaces can opt out on a case-by-case basis with the standard ! or - prefix.

Phase 4 self-review (five-axis)

Correctness: No finding — single yaml line addition; the plugin name string is the canonical Gitea repo name.
Readability: No finding — comment matches the shape of the surrounding plugins.
Architecture: No finding — defaults.plugins is the canonical place to add cross-cutting skills.
Security: No finding — no new code path; plugin install honors the existing trust model.
Performance: No finding — plugins load is amortized at workspace boot.

Refs

Wires the new SOP Phase 4 self-review skill into `dev-department.yaml` defaults so every dev workspace (Dev Lead, Core Platform, Controlplane, App-Docs, Infra, SDK + floaters + Triage Operator) picks it up automatically. ## What ```diff plugins: - molecule-skill-cron-learnings - molecule-skill-update-docs + - molecule-skill-five-axis-review # SOP Phase 4 — structured five-axis self-review before PR ``` ## Why Five-Axis self-review on the `!external` resolver work (internal#77, PRs #105+#106 of molecule-core) caught three Required-severity issues that the same author's unstructured 3-weakest pass on the same code missed: | Issue | Caught by | |---|---| | Cache-validity gap | Correctness axis | | Token persistence in `.git/config` | Security axis | | Misleading function name | Readability axis | Setting it as a default rather than per-workspace because every agent doing substantive change benefits from the structured pass — it's the default mode of operation, not an opt-in. ## Plugin contract Per org-template plugin resolution (PR #71, issue #68 of molecule-core) the `defaults.plugins` UNIONs with per-workspace `plugins:` lists. Workspaces can opt out on a case-by-case basis with the standard `!` or `-` prefix. ## Phase 4 self-review (five-axis) **Correctness:** No finding — single yaml line addition; the plugin name string is the canonical Gitea repo name. **Readability:** No finding — comment matches the shape of the surrounding plugins. **Architecture:** No finding — defaults.plugins is the canonical place to add cross-cutting skills. **Security:** No finding — no new code path; plugin install honors the existing trust model. **Performance:** No finding — plugins load is amortized at workspace boot. ## Refs - internal#84 — canonical SOP doc with new Phase 4 wording - molecule-core PR #107 — hardening pass driven by the five-axis review - Plugin tag v1.0.0 (sha `4835359`) — https://git.moleculesai.app/molecule-ai/molecule-ai-plugin-molecule-skill-five-axis-review/src/tag/v1.0.0
claude-ceo-assistant added 1 commit 2026-05-08 13:08:34 +00:00 
Wire the new SOP Phase 4 self-review skill into dev-department defaults
so every dev workspace (Dev Lead, Core Platform, Controlplane, App-Docs,
Infra, SDK + floaters + Triage Operator) picks it up automatically.

The plugin (v1.0.0,
git.moleculesai.app/molecule-ai/molecule-ai-plugin-molecule-skill-five-axis-review)
implements the structured Five-Axis self-review that replaces the
unstructured 'list 3 weakest spots' Phase 4 (canonical SOP doc:
internal/runbooks/dev-sop.md, opened in internal#84).

Per the org-template plugin resolution contract (PR #71, issue #68),
this UNIONs with per-workspace plugins lists. Workspaces can opt out
on a case-by-case basis with the standard '!molecule-skill-five-axis-review'
or '-molecule-skill-five-axis-review' prefix in their own plugins: list.

WHY
  Five-Axis self-review on the !external resolver work (internal#77,
  PRs #105+#106 of molecule-core) caught three Required-severity issues
  that the same author's unstructured 3-weakest pass on the same code
  missed: cache validity gap, token persistence in .git/config, misleading
  function name post-refactor. The structured pass on PR #107 caught all
  three on first read.

  Setting the skill as a default rather than per-workspace because every
  agent doing substantive change benefits from the structured pass — it's
  the default mode of operation, not an opt-in.

REFS
  internal#84 — canonical SOP doc with new Phase 4 wording
  molecule-core PR #107 — hardening pass driven by the five-axis review
  Plugin tag v1.0.0 sha 4835359

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/add-five-axis-review-plugin:feat/add-five-axis-review-plugin
git checkout feat/add-five-axis-review-plugin
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: molecule-ai/molecule-dev-department#6
No description provided.
Delete Branch "feat/add-five-axis-review-plugin"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?