Skip to content

chore(repo): remove internal process docs and dead code from the tree#45

Merged
aykhan019 merged 1 commit into
aykhan019:mainfrom
endorphin13:chore/clean-submission-tree
May 31, 2026
Merged

chore(repo): remove internal process docs and dead code from the tree#45
aykhan019 merged 1 commit into
aykhan019:mainfrom
endorphin13:chore/clean-submission-tree

Conversation

@endorphin13

Copy link
Copy Markdown
Contributor

Why

Several internal coordination artifacts were committed into the project tree. They are not part of the application, and one of them (HANDOFF.md + scripts/commit-as.sh) encoded commit-attribution and tooling-disclosure rules that should not govern the repository or a graded submission.

What changed

Removed

  • HANDOFF.md, CHECKLIST.md, MERGE_PLAN.md — session/planning notes.
  • scripts/commit-as.sh and scripts/.authors — the per-commit author-rotation wrapper.
  • CLAUDE.md — empty (0 bytes).
  • integration/ — a one-time visual drop already merged into apps/web; it sat outside the pnpm workspace globs yet was still linted/formatted by CI.

Updated to drop references to the above

  • CONTRIBUTING.md — removed the commit-as.sh mandate, the HANDOFF/CHECKLIST workflow, and the "no external tool/AI names" rule; kept branching, commit format, code standards, testing, security.
  • .github/pull_request_template.md, .github/CODEOWNERS.
  • docs/adr/0001 — also corrected the TanStack Query claim (it is not installed; the web app uses the fetch-based api-client), and docs/adr/0002 + docs/design/explorations.md.

Other

  • .gitignore now ignores the two course handout PDFs so they are not accidentally committed.

No application code is touched; apps/ and packages/ are unchanged, so the build and tests are unaffected.

These internal coordination artifacts do not belong in the project source,
and one of them encoded commit-attribution and tooling-disclosure rules that
should not govern the repository.

- Remove HANDOFF.md, CHECKLIST.md, MERGE_PLAN.md (session/planning notes).
- Remove scripts/commit-as.sh and scripts/.authors (the per-commit author
  rotation wrapper) and every reference to it.
- Remove the empty CLAUDE.md.
- Remove the dead integration/ tree: a one-time visual drop already merged
  into apps/web, sitting outside the pnpm workspace yet still linted by CI.
- Rewrite CONTRIBUTING.md, the PR template, and CODEOWNERS to drop the
  commit-as.sh / HANDOFF / CHECKLIST workflow.
- Fix ADR-0001's TanStack Query claim (not installed; the web app uses a
  fetch-based api-client) and drop commit-as.sh mentions from ADR-0001/0002
  and the design explorations note.
- Gitignore the two course handout PDFs so they are not accidentally committed.
@endorphin13 endorphin13 requested a review from aykhan019 as a code owner May 31, 2026 17:57
@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown

@fateh-mammadli is attempting to deploy a commit to the Aykhan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@aykhan019 aykhan019 merged commit b75785a into aykhan019:main May 31, 2026
1 of 2 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.

3 participants