Mandatory inline result display at end of dev loop (#190)#191
Merged
Conversation
added 5 commits
June 7, 2026 09:35
Contributor
Author
Evidence: mandatory inline result display (#190)Change type: CLI / PowerShell helper. Artifact: markdown (Inline). Intent (issue #190)End-of-cycle result display is mandatory. Inline (.md/CLI/text) artifacts render A. Inline echo (default), ANSI strippedCommand: Captured output (note the link line AND the echoed content; the source artifact B. -SkipDisplay: link only, no inline echoCommand: Captured output (no content echo, link still printed): C. ArtifactReference (.html): link onlyCommand: Captured output (no raw HTML echoed; link only): Test evidenceAI review
Result: PASSED. |
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
Makes the end-of-cycle result display mandatory and self-enforcing so the
user can confirm a change worked without re-running the command.
Publish-Evidence.ps1now echoes the artifact content inline for Inline(
.md/ CLI / text) artifacts, with ANSI escape sequences stripped so thefenced block is clean plain text. The echo runs in both
-LocalOnlyand normalmodes. ArtifactReference (UI HTML+video, binary) artifacts stay link-only.
A new
-SkipDisplayswitch (an explicit, user-approved opt-out) suppresses theinline echo while still printing the
Evidence (local): file:///link line.The instruction set is updated so the display is a required, gate-enforced field
of the Task Complete Summary and is never silently skipped by the agent.
Closes #190.
Changes
helpers/Publish-Evidence.ps1: inline ANSI-stripped content echo for Inlineartifacts (both modes); new
-SkipDisplayswitch; empty-file null guard.tests/Publish-Evidence.Tests.ps1: behavior-first tests (inline contentechoed; ANSI stripped; ArtifactReference not echoed;
-SkipDisplaylink-only).evidence-capture/SKILL.md: "Result display contract" + compliance checklistitem + documented opt-out.
dev-loop.agent.md: Phase 5b mandatory "Display the result" step; hardened"hard gate, never skipped by the agent" wording; opt-out mapping; updated
"When the Loop Is Complete".
dev-loop-phase-gate/SKILL.md: after-Phase-5b gate item for the displayedresult.
copilot-instructions.mdandCLAUDE.md: mandatory Result display field inthe Task Complete Summary; removed the soft "omit" escape for it.
Behavior-first
The helper change shipped test-first: the four new tests were observed RED
before implementation, then GREEN.
Test
Full
.githubPester suite: 371 passed, 0 failed, 4 skipped.