Skip to content

Mandatory inline result display at end of dev loop (#190)#191

Merged
MarkMichaelis merged 5 commits into
mainfrom
feat/190-display-result
Jun 7, 2026
Merged

Mandatory inline result display at end of dev loop (#190)#191
MarkMichaelis merged 5 commits into
mainfrom
feat/190-display-result

Conversation

@MarkMichaelis

Copy link
Copy Markdown
Contributor

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.ps1 now echoes the artifact content inline for Inline
(.md / CLI / text) artifacts, with ANSI escape sequences stripped so the
fenced block is clean plain text. The echo runs in both -LocalOnly and normal
modes. ArtifactReference (UI HTML+video, binary) artifacts stay link-only.

A new -SkipDisplay switch (an explicit, user-approved opt-out) suppresses the
inline 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 Inline
    artifacts (both modes); new -SkipDisplay switch; empty-file null guard.
  • tests/Publish-Evidence.Tests.ps1: behavior-first tests (inline content
    echoed; ANSI stripped; ArtifactReference not echoed; -SkipDisplay link-only).
  • evidence-capture/SKILL.md: "Result display contract" + compliance checklist
    item + 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 displayed
    result.
  • copilot-instructions.md and CLAUDE.md: mandatory Result display field in
    the 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

Invoke-Pester -Path .github/skills/evidence-capture/tests/

Full .github Pester suite: 371 passed, 0 failed, 4 skipped.

@MarkMichaelis

Copy link
Copy Markdown
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
their real result inline with ANSI stripped; -SkipDisplay suppresses the echo
but keeps the Evidence (local): link; ArtifactReference (UI/binary) stays
link-only.

A. Inline echo (default), ANSI stripped

Command:

Publish-Evidence.ps1 -ArtifactPath sample-artifact.md -PullRequest 1 -LocalOnly

Captured output (note the link line AND the echoed content; the source artifact
contained ESC[32m...ESC[0m color codes which are stripped):

Evidence (local): file:///.../sample-artifact.md
----- Evidence (inline result) -----
# Evidence: inline result display ...
PASS all 20 tests green
The widget now returns 42.
----- end evidence -----

B. -SkipDisplay: link only, no inline echo

Command:

Publish-Evidence.ps1 -ArtifactPath sample-artifact.md -PullRequest 1 -LocalOnly -SkipDisplay

Captured output (no content echo, link still printed):

Evidence (local): file:///.../sample-artifact.md

C. ArtifactReference (.html): link only

Command:

Publish-Evidence.ps1 -ArtifactPath ui.html -PullRequest 1 -LocalOnly

Captured output (no raw HTML echoed; link only):

Evidence (local): file:///.../ui.html

Test evidence

Invoke-Pester -Path .github/skills/evidence-capture/tests/
Tests Passed: 20, Failed: 0

AI review

  • A. Artifact visibly confirms the intent given the diff: YES -- all three
    documented behaviors (inline ANSI-stripped echo, -SkipDisplay link-only,
    ArtifactReference link-only) are demonstrated by real runs.
  • B. New problem introduced by the diff: NONE -- the link line still prints in
    every mode; full Pester suite green.

Result: PASSED.

@MarkMichaelis MarkMichaelis merged commit a65399c into main Jun 7, 2026
2 checks passed
@MarkMichaelis MarkMichaelis deleted the feat/190-display-result branch June 7, 2026 16:49
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.

Dev loop must reliably DISPLAY the result at end of cycle (inline for CLI/markdown, link for UI/binary)

1 participant