Skip to content

chore: prepare the repo for public use - #104

Merged
chriskehayias merged 1 commit into
mainfrom
chore/public-repo-cleanup
Sep 29, 2026
Merged

chriskehayias merged 1 commit into
mainfrom
chore/public-repo-cleanup

Conversation

@chriskehayias

Copy link
Copy Markdown
Contributor

Summary

Public-repo hygiene pass, from a full documentation and repository review.

  • License: add MIT LICENSE; set license, repository, homepage, bugs in package.json; README License section now links it. "private": true stays (it only blocks npm publish).
  • Placeholder host: mpi.ministryplatform.com → your-instance.ministryplatform.com in .env.example and scripts/setup.ts. Setup no longer offers the placeholder as the default host, so pressing Enter can't save a non-working value.
  • Internal files removed: the three dated .claude/reports/deps-audit-*.md (now a gitignored local copy; deps-known-issues.md remains the durable record, and /audit-deps is updated accordingly) and the open setup-check TODO.
  • Review record moved: .claude/TODO/security-review-2026-09-28-index.md → docs/security/2026-09-28-auth-review.md, with instance-specific wording neutralized.
  • Test comments: comments that cited deleted .claude/TODO/ files now name the fix; TODO security-* tags reworded as review items so closed work doesn't read as open.
  • .gitignore: .claude/reports/, /out/, npm-debug.log*, .DS_Store, Thumbs.db, .idea/.

No secrets were found in tracked files or history. No runtime code changes other than the setup prompt default.

Test plan

  • npx vitest run — 85 files, 2034 passed / 3 skipped
  • npm run lint, npx tsc --noEmit
  • npm run deps:verify (lockfile gained only the root "license": "MIT")

🤖 Generated with Claude Code

- Add an MIT LICENSE; set license/repository/homepage/bugs in package.json
  and replace the README's "License: Private".
- Replace the real-looking mpi.ministryplatform.com default with the
  your-instance.ministryplatform.com placeholder in .env.example and
  setup.ts, and stop setup offering the placeholder as the host default.
- Remove internal working files: the dated deps-audit reports (now
  gitignored local copies; deps-known-issues.md is the durable record)
  and the open setup-check TODO.
- Move the closed 2026-09-28 auth review record from .claude/TODO/ to
  docs/security/2026-09-28-auth-review.md and neutralize its
  instance-specific wording.
- Repoint test comments that cited deleted TODO files, and reword
  "TODO security-*" tags as review items so they don't read as open work.
- .gitignore: .claude/reports/, /out/, npm-debug.log*, OS/editor files.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Sep 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@chriskehayias
chriskehayias merged commit bc104e1 into main Sep 29, 2026
5 checks passed
@chriskehayias
chriskehayias deleted the chore/public-repo-cleanup branch September 29, 2026 22:24
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