FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: block ineffective interactive artifacts by ramsani · Pull Request #40 · ramsani/html-explainer · GitHub

This repository was archived by the owner on Jun 12, 2026. It is now read-only.
/ html-explainer Public archive

test: block ineffective interactive artifacts - #40

Merged
ramsani merged 4 commits into
mainfrom
product/semantic-effectiveness-audit
May 22, 2026
Merged

test: block ineffective interactive artifacts#40
ramsani merged 4 commits into
mainfrom
product/semantic-effectiveness-audit

Conversation

ramsani commented May 22, 2026

Copy link
Copy Markdown
Owner

Summary

Adds semantic hard-fails for artifacts that are structurally complete but ineffective.

Why

The repo now has interactive capabilities and routing. This PR prevents regressions where an artifact looks complete but does not state the work it removes, does not map evidence to consequence, or declares interaction without real controls plus reusable output.

Changes

  • Adds hard-fails for missing effectiveness job, missing evidence-to-consequence mapping, and ineffective interactive capabilities.
  • Adds tests/artifacts/bad/complete-but-ineffective.html.
  • Updates tests/artifacts/good/interactive-tool.html so its evidence has a consequence/action link.
  • Confirms merged interactive templates still pass audit-artifact.py --min-score 90.

Validation

  • Bad fixture blocks as expected.
  • scripts/validate-golden-artifacts.sh
  • scripts/validate-runtime.sh
  • Individual audit of merged templates under templates/.

Rollback

Revert this PR to remove the stricter semantic hard-fails and fixture changes.

ramsani merged commit d1a3297 into main May 22, 2026
1 check passed
ramsani deleted the product/semantic-effectiveness-audit branch May 22, 2026 05:58
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL