Skip to content

chore: remove internal artifacts and simplify repository tooling#15

Merged
EddieTYP merged 4 commits into
mainfrom
codex/repo-content-cleanup
Jul 16, 2026
Merged

chore: remove internal artifacts and simplify repository tooling#15
EddieTYP merged 4 commits into
mainfrom
codex/repo-content-cleanup

Conversation

@EddieTYP

Copy link
Copy Markdown
Owner

What changed

  • remove tracked codebase-memory, generated agent reports, implementation plans, QA notes, audit-only screenshots, duplicate assets, and redundant placeholders
  • add repository guardrails through .gitignore, AGENTS.md, contributor guidance, and a CI test that rejects tracked local/ignored artifacts
  • replace the 1,730-line frontend source-string test suite with focused utility behavior tests plus TypeScript/Vite builds
  • remove unused frontend dependencies and legacy versioned demo scripts
  • simplify GitHub Pages to build and deploy only the current read-only demo
  • sync Traditional and Simplified Chinese release links to v0.8.0
  • isolate release-package test outputs in pytest temp directories

Why

Generated development metadata and speculative planning material had been committed alongside product code. The Pages workflow and static frontend tests also accumulated historical complexity that was not needed by current users. Release tests additionally left hundreds of megabytes of ignored artifacts in the repository root.

Impact

The current product behavior remains unchanged. The repository is smaller, current-only deployment is easier to maintain, accidental local artifact commits now fail CI, and installer tests no longer pollute the checkout.

Validation

Passed locally on Windows:

  • npm run test:frontend — 4 passed
  • local TypeScript/Vite build — passed
  • demo-mode TypeScript/Vite build — passed
  • tests/test_github_pages_demo.py plus Windows README contract — 5 passed
  • tests/test_public_mvp.py — 14 passed
  • tests/test_installer_release.py — 13 passed; dist-release remained absent
  • username-derived metadata and stale deleted-file scans — clean
  • git diff --check — passed

Full local pytest result: 418 passed, 4 skipped, 12 failed. The 12 failures are existing Windows-platform assumptions in untouched areas: file-URI handling, POSIX path separator expectations, symlink privilege requirements, and missing Unix command-line tools. GitHub Actions remains the cross-platform release gate.

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 94056dffd3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/pages.yml

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🎉

Reviewed commit: f0599622e7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@EddieTYP
EddieTYP marked this pull request as ready for review July 16, 2026 08:21
@EddieTYP
EddieTYP merged commit 06350df into main Jul 16, 2026
2 checks passed
@EddieTYP
EddieTYP deleted the codex/repo-content-cleanup branch July 18, 2026 16:35
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