Skip to content

Implementation Plan: Remove Redundant Planning-Stage Mermaid Generation#4331

Merged
Trecek merged 5 commits into
developfrom
impl-remove_redundant_plan_mermaid_plan_2026-07-22_130742-20260722-163424
Jul 23, 2026
Merged

Implementation Plan: Remove Redundant Planning-Stage Mermaid Generation#4331
Trecek merged 5 commits into
developfrom
impl-remove_redundant_plan_mermaid_plan_2026-07-22_130742-20260722-163424

Conversation

@Trecek

@Trecek Trecek commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Remove the dead architecture-lens workflow from the make-plan and rectify skills so planning sessions no longer select a lens, write an arch_lens_selection_* rationale artifact, load arch-lens/mermaid, or embed ## Proposed Architecture in plan templates. Preserve the general Foundation Audit, Interface Mapping, Registry Trace, and complexity-gating behavior because those reviewers analyze the full plan rather than the discarded diagram.

Implementation Plan

Plan file: /home/talon/projects/generic_automation_mcp/.autoskillit/temp/make-plan/remove_redundant_plan_mermaid_plan_2026-07-22_130742.md

Closes #4310

🤖 Generated with Claude Code via AutoSkillit

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: approved_with_comments

Comment thread src/autoskillit/skills_extended/make-plan/SKILL.md
Comment thread tests/skills/test_adversarial_review_contracts.py Outdated

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.

@Trecek
Trecek added this pull request to the merge queue Jul 23, 2026
Merged via the queue into develop with commit 80572ca Jul 23, 2026
3 checks passed
@Trecek
Trecek deleted the impl-remove_redundant_plan_mermaid_plan_2026-07-22_130742-20260722-163424 branch July 23, 2026 01:39
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