Repository navigation
Refresh homepage for current factory workflows - #665
Merged
Merged
Conversation
Merged
This was referenced Oct 19, 2026
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
Refresh the homepage from the current README and domain model. The press hall now links to installed workflows, and the surrounding copy covers Harness choices, Claims, setup/update, scheduling and optional Security review. Mark the #24 replay as historical.
Share decorative press and chest geometry once; keep each line's text local.
Closes #663
Evidence
Before: The homepage browser check against
mainfailed:all six current workflows must appear,4 !== 6; three lines saidNOT YET INSTALLED.After: The same focused check passes at 1280, 390 and 320 pixels with JavaScript enabled and disabled: six installed lines, workflow copy, README links, unclipped SVG text, no horizontal overflow and keyboard focus. Desktop and phone screenshots inspected after the Standards fix.
cargo check --all-targets,cargo fmt --check, andcargo clippy --all-targets -- -D warningspassed.Existing
share_card(4 tests) andsite_deploy(7 tests) integration tests passed.cargo testpassed for the full suite (exit 0); log retained at/tmp/issue-663-cargo-test.log.Validation seams
Review
Reviewed against
main(cfda566) with independent Standards and Spec reviewers. Standards: one P3 duplication judgment call, addressed by sharing both chest rectangles. Spec: no findings. No behavior findings were raised.Both axes read all changed files (
site/index.html,site/styles.css). Changed files left unread: none. Reports, including each axis's final files-read list, are retained in.thirdshift-review-ZgRuM9/standards.mdand.thirdshift-review-ZgRuM9/spec.mdin the Run worktree.Unaddressed findings
Standards
None.
Spec
None.
Security
None. Guidance-mode review of
git diff cfda566cc9b40f5fb9e1c601b533560329577b7c...HEADatc16e780218ea32f0577e29fcee84e19b46bc29b2found no introduced Security findings. Read both changed files and the supporting JavaScript, Caddy configuration, publisher, systemd units, share-card renderer and relevant tests, using the injection, client-side and HTTP security guidance. No repositorySECURITY.mdor threat model was present.Offline structural checks passed on HEAD and the pinned merge base: fixed HTTPS/local URLs, unique IDs, local SVG references, no inline event handlers or form/frame/object/embed elements, and only the existing module script. Supporting executable/configuration files are unchanged. No candidate required a proof-of-concept or a fix; no target code or deployed service was executed.
Security review outcome
No unaddressed introduced findings.
Merge Danger
Door: two-way
Revert the site files to roll back. The existing publisher serves these changes after they land on
main.Blast Radius: homepage
Static homepage copy and floor-plan styling; no factory runtime changes.
Built with codex · gpt-6.1-sol · high