Skip to content

docs: remove binary assets and document screenshot capture#6

Merged
Wasif-ZA merged 1 commit into
mainfrom
codex/implement-repository-cleanup-specifications
Jan 24, 2026
Merged

docs: remove binary assets and document screenshot capture#6
Wasif-ZA merged 1 commit into
mainfrom
codex/implement-repository-cleanup-specifications

Conversation

@Wasif-ZA

Copy link
Copy Markdown
Owner

Motivation

  • Binary image files under docs/assets/ were not appropriate for this environment and should not be committed.
  • Preserve the documented screenshot workflow and expected filenames so README references remain accurate.

Description

  • Deleted committed binary assets docs/assets/architecture-placeholder.png, docs/assets/screenshot-home.png, and docs/assets/screenshot-feature.png.
  • Added docs/assets/README.md that documents copy/paste steps to capture and place screenshots locally.
  • Updated README.md to stop embedding images and instead reference the expected docs/assets/ filenames and point to the new capture guide.

Testing

  • Ran cd ui && npm run lint, which completed (ESLint emitted warnings only).
  • Ran cd ui && npm run test, which executed the smoke test ui/tests/smoke.test.mjs and passed.
  • Ran cd ui && npm run build, which completed successfully.

Codex Task

@vercel

vercel Bot commented Jan 24, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
bridge-opening-project Ready Ready Preview, Comment Jan 24, 2026 5:41am

@Wasif-ZA
Wasif-ZA merged commit 8c8fd03 into main Jan 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant