Feature/concise mode - #8
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces the new concise runtime response-style skill and performs a broad template refactor to standardize agent structure, reduce duplicated procedural guidance, and clarify artifact ownership/handoffs across the vstack role/skill system.
Changes:
- Added the
conciseskill (templates, config, CLI canonical name wiring) and referenced it from all six role agents with per-role defaults. - Standardized agent templates around a shared structure and moved procedural detail into skills, including explicit gate moments/handoffs.
- Added/normalized “Deliverable and artifact policy” sections and expanded verification observability checks; regenerated
.github/artifacts and updated docs/README/CHANGELOG.
Reviewed changes
Copilot reviewed 45 out of 45 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/vstack/agents/test_role_wiring.py | Adds regression coverage ensuring all role templates reference @#concise. |
| tests/conftest.py | Removes duplicated canonical skill list (but currently breaks downstream imports/tests). |
| src/vstack/cli/constants.py | Adds concise to the canonical skill name list. |
| src/vstack/_templates/skills/vision/template.md | Adds deliverable/artifact policy section. |
| src/vstack/_templates/skills/verify/template.md | Adds deliverable/artifact policy and observability evidence requirements (scope-dependent). |
| src/vstack/_templates/skills/security/template.md | Adds deliverable/artifact policy section. |
| src/vstack/_templates/skills/requirements/template.md | Adds deliverable/artifact policy section. |
| src/vstack/_templates/skills/release-notes/template.md | Clarifies ownership of release artifacts and adds deliverable/artifact policy + scope-conditional checks. |
| src/vstack/_templates/skills/pr/template.md | Adds deliverable/artifact policy and PR body sourcing guidance. |
| src/vstack/_templates/skills/performance/template.md | Adds deliverable/artifact policy section. |
| src/vstack/_templates/skills/inspect/template.md | Adds deliverable/artifact policy and observability/reliability checks section. |
| src/vstack/_templates/skills/docs/template.md | Adds deliverable/artifact policy section. |
| src/vstack/_templates/skills/design/template.md | Adds deliverable/artifact policy and minor formatting cleanup. |
| src/vstack/_templates/skills/concise/template.md | Introduces the concise skill body defining modes, precedence, and expected responses. |
| src/vstack/_templates/skills/concise/config.yaml | Adds concise skill metadata/config for generation and slash-command UX. |
| src/vstack/_templates/skills/architecture/template.md | Adds deliverable/artifact policy and ADR recording guidance. |
| src/vstack/_templates/agents/tester/template.md | Refactors tester agent to standardized structure; adds @#concise and scope-conditional perf baseline language. |
| src/vstack/_templates/agents/release/template.md | Refactors release agent to standardized structure; clarifies gates/handoffs and artifact checks. |
| src/vstack/_templates/agents/product/template.md | Refactors product agent to standardized structure; adds @#concise and explicit gate moments. |
| src/vstack/_templates/agents/engineer/template.md | Refactors engineer agent to standardized structure; adds @#concise and streamlined “how you work”. |
| src/vstack/_templates/agents/designer/template.md | Refactors designer agent to standardized structure; adds @#concise and explicit gate moments/handoffs. |
| src/vstack/_templates/agents/architect/template.md | Refactors architect agent to standardized structure; adds @#concise and explicit gates/handoffs. |
| src/vstack/_templates/agents/_partials/agent-skill-boundary.md | Adds shared agent-skill boundary partial for reuse across role templates. |
| docs/design/skills.md | Adds concise to the skill index table and keeps skill invocation contract documented. |
| README.md | Updates role table with default concise modes and documents concise usage. |
| CHANGELOG.md | Adds 1.1.0 release notes describing concise + refactor. |
| .github/vstack.json | Updates generated install manifest metadata and adds concise artifact entry. |
| .github/skills/vision/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/verify/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/security/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/requirements/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/release-notes/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/pr/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/performance/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/inspect/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/docs/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/design/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/skills/concise/SKILL.md | New generated skill output for concise. |
| .github/skills/architecture/SKILL.md | Regenerated skill output reflecting template updates. |
| .github/agents/tester.agent.md | Regenerated agent output reflecting template refactor + concise. |
| .github/agents/release.agent.md | Regenerated agent output reflecting template refactor + concise. |
| .github/agents/product.agent.md | Regenerated agent output reflecting template refactor + concise. |
| .github/agents/engineer.agent.md | Regenerated agent output reflecting template refactor + concise. |
| .github/agents/designer.agent.md | Regenerated agent output reflecting template refactor + concise. |
| .github/agents/architect.agent.md | Regenerated agent output reflecting template refactor + concise. |
eschaar
force-pushed
the
feature/concise-mode
branch
from
April 19, 2026 22:32
6fb487a to
d44d191
Compare
eschaar
force-pushed
the
feature/concise-mode
branch
from
April 19, 2026 22:43
d44d191 to
e5fb821
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implemented a comprehensive prompt-architecture cleanup and consistency pass across roles and skills.
This PR combines the concise-mode feature work with a full agent/skill boundary refactor to reduce template size, remove duplicated procedural guidance, and make role handoffs/gates explicit and consistent.
Key outcomes:
conciseskill across all roles with per-role defaults.Deliverable and artifact policysections across relevant skills.release-notesowns bothdocs/releases/{date}.mdandCHANGELOG.md;prowns push/PR creation..githubartifacts from template source of truth.Net effect: clearer role boundaries, stronger agent↔skill contracts, lower prompt footprint, and a single coherent 1.1.0 release scope.
Related Issues
docs/architecture/adr/*remain applicable; no new ADR required for this templating-only restructuring.Validation
Validation notes:
188 passed100%Release Impact
feat:(minor)fix:(patch)BREAKING CHANGE(major)