Skip to content

Add screenshot handoff between UX and UI design workflows - #159

Closed
redhat-chai-bot wants to merge 42 commits into
flightctl:mainfrom
redhat-chai-bot:ux-ui-screenshot-handoff
Closed

redhat-chai-bot wants to merge 42 commits into
flightctl:mainfrom
redhat-chai-bot:ux-ui-screenshot-handoff

Conversation

@redhat-chai-bot

@redhat-chai-bot redhat-chai-bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Include curated, revision-bound prototype screenshots beside UX flows and states, with captions and a sanitized published identity manifest.
  • Publish assets using an explicit repository-selected Git, existing LFS, or external-reference policy; do not select ordinary Git implicitly or enable LFS automatically.
  • Reuse an owned-path staging/commit procedure for initial publication and updates, including image-only changes and final-image removal, while preserving unrelated staged and working-tree changes.
  • Require UI ingestion to open supplied images and retain per-image identity, observations, availability, and acceptance decisions. Use that visual evidence in UI planning and flag contradictions with the written specification.
  • Add regression coverage and CI discovery, update documentation, and bump ux-design to 0.6.0 and ui-design to 0.2.0.

Validation

  • Markdownlint: 328 files checked, no issues.
  • Screenshot lifecycle suite: 22 tests passed, including executable tests of the extracted Bash staging/commit recipe in temporary Git repositories and structural contract checks.
  • Repository version validation, global pre-review checks, and CI-pinned Skillsaw completed successfully. Existing shared/UI unit suites also passed.
  • Full eight-dimension reviews of both workflows found no blocking introduced issues.
  • Existing UI per-workflow orphan-reference/sync-size diagnostics and offline link-check template-placeholder diagnostics remain documented baseline findings.

Verification scope

The executable regression tests exercise owned-path staging and commits; they do not execute the prose-only external asset retrieval/reference-rewrite procedure or a complete live prototype-to-UX-to-UI/Jira workflow. A full live end-to-end run remains to be performed. No screenshot binaries or external storage provider integration are added by this PR.


AI-generated. Review for accuracy.

@adalton requested from Slack

Summary

This PR adds a revision-bound screenshot handoff between the ux-design and ui-design packages. It updates workflow instructions and guidelines, adds two shared recipes, adds screenshot lifecycle tests, and extends CI test discovery.

ux-design

  • /evaluate now records screenshots captured or accepted in the current run, with SHA-256 hashes and context, discovery, and prototype revision values.
  • /handoff now curates screenshots, assigns stable SS-N IDs, and records captions, alt text, checksums, and revision metadata. It supports an explicitly authorized text-only handoff.
  • /publish, /respond, and /revise now handle screenshot files, manifests, references, storage policy, and removals alongside handoff text. Publish and response flows use an owned-path staging and commit procedure, validate push destinations, and record pending attempts for recovery.
  • Guidelines distinguish private raw evaluation screenshots from curated handoff images and require authorization before copying, committing, or uploading screenshots.
  • The package version changes from 0.5.0 to 0.6.0.

ui-design

  • /ingest now checks screenshot provenance and uses the shared inspection recipe to match indexed IDs to embeds, resolve images, verify identity and revision bindings, and record observations and availability.
  • /plan now uses eligible screenshot observations for visual decisions, flags conflicts with written specifications, and notes when visual evidence is deferred or unavailable. Screenshots do not establish accessibility, API, or runtime behavior.
  • Guidelines prohibit fabricated visual observations when images cannot be inspected.
  • The package version changes from 0.1.0 to 0.2.0.

Shared resources and CI

  • _shared/recipes/screenshot-inspection.md defines screenshot matching, path containment, identity and revision checks, inspection outcomes, and citation eligibility.
  • _shared/recipes/screenshot-publish-stage.md defines owned-path staging, commits, and push-recovery handling for UX publishing and responses.
  • AGENTS.md adds both recipes to the shared recipe inventory.
  • CI now discovers tests under ui-design/scripts and ux-design/scripts. This PR adds screenshot lifecycle tests under ux-design/scripts.

The changes establish documentation and workflow procedures. They do not add screenshot binaries or an external storage provider integration.

Loading
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