Skip to content

Feat/hac 278 tally showcase - #7

Merged
qmarcelle merged 3 commits into
mainfrom
feat/hac-278-tally-showcase
Jul 30, 2026
Merged

Feat/hac 278 tally showcase#7
qmarcelle merged 3 commits into
mainfrom
feat/hac-278-tally-showcase

Conversation

@qmarcelle

Copy link
Copy Markdown
Contributor

No description provided.

Use tally_impact_screen.png (1920x1275) as the cockpit preview image
instead of the explainer01 diagram. Remove the placeholder caption
that said a screenshot would replace the diagram.
- Move screenshot from public/ to src/assets/tally-impact-hero.png
- Import via astro:assets Image component (not plain <img>)
- Use priority prop for eager loading + fetchpriority=high (LCP)
- Preserve native 1920x1275 aspect ratio with object-fit: contain
- Responsive srcset: 768/1280/1600/1920 widths, PNG format
- Specific alt text: 'Tally Impact review for game_events showing
  exact source resolution, three declared evidence gaps, and
  upstream and downstream lineage.'
- Adjacent conclusion text states the important result
- Image links to interactive cockpit (liveEvidence URL)
- 'View full-resolution screenshot' link below image
- Remove overlay badge, browser-dot mockup, placeholder caption
- 1px graphite border, 12px radius, subtle shadow
- Container max-width increased to 1400px (spec: ~1280-1440px)
- Add tally-impact-hero.png to source lock with SHA-256
- Update verifier: check astro:assets import and src/assets path
- Update tests: 10 new product screenshot assertions
- Visual regression captured at 1440x900, 1280x800, 768x1024, 390x844
Copilot AI review requested due to automatic review settings July 30, 2026 12:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

…As to live app

- Revert blanket org rename: publication provenance stays at workspace-json/
  for agents-audit and codex-mcp; current standard/governance/contribution
  links point to workspacejson/standard (canonical authority per OWNERSHIP.md)
- Restructure contract.json tools: add canonicalRepository,
  publicationRepository, releaseRef, releaseCommit, authorityStatus,
  authorityEvidence for each tool
- Use developmentRepository + authorityStatus: "transitional" for codex-mcp
  (workspacejson/integrations has no OWNERSHIP.md asserting canonical authority)
- Add @workspacejson/cli@0.5.2 to contract, implementations, changelog,
  getting-started page
- Create sources/ecosystem-packages.json with all 5 npm packages, resolved
  releaseCommit SHAs, authority evidence from OWNERSHIP.md
- Create scripts/verify-ecosystem.mjs: offline consistency + live npm/GitHub
  verification (tag dereferencing, commit resolution, dist-tag matching)
- Wire verify:ecosystem into build chain (schema:verify + content:verify +
  verify:ecosystem + astro build)
- Add .github/workflows/ecosystem-freshness.yml: nightly schedule,
  workflow_dispatch, repository_dispatch for live freshness checks
- Route Tally showcase CTAs to https://tally.workspacejson.dev/ (live app)
- Add SEO/AISEO meta tags to Tally showcase: og:site_name, og:url,
  twitter:card, twitter:title, twitter:description, robots, canonical
- Enrich Tally JSON-LD: installUrl, downloadUrl, applicationSubCategory,
  offers, publisher
- Re-capture homepage example with agents-audit@0.4.4 on 2026-07-30
  (fixes stale timestamp from 0.4.3 capture)
- Update spec.mdx JSON-LD dateModified to 2026-07-30
- Remove redundant releaseRef from contract schema source (ref already used)
- Remove no-op .replace("/", "/") in verify-ecosystem.mjs
- Update tests: Try Tally CTA, JSON-LD url field, contract schema source
  field rename
@qmarcelle
qmarcelle force-pushed the feat/hac-278-tally-showcase branch from faef7ed to b5224b5 Compare July 30, 2026 13:04
@qmarcelle
qmarcelle merged commit 46d1d78 into main Jul 30, 2026
4 checks passed
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.

2 participants