Skip to content

Remove accidental Unstuck subproject#70

Merged
simongonzalezdc merged 1 commit into
mainfrom
codex/remove-ef-coach-subproject
Jun 3, 2026
Merged

Remove accidental Unstuck subproject#70
simongonzalezdc merged 1 commit into
mainfrom
codex/remove-ef-coach-subproject

Conversation

@simongonzalezdc
Copy link
Copy Markdown
Member

@simongonzalezdc simongonzalezdc commented Jun 3, 2026

Summary

  • removes the former EF-COACH/ Unstuck subtree from Dev Learning Archaeologist
  • keeps the root test command scoped to Dev Learning Archaeologist only
  • adds docs/project-separation.md so Unstuck work stays in its own repos

Verification

  • npm test

Repo split

  • Live Unstuck source now lives in KyaniteLabs/unstuck-coach-live
  • Protocol/docs source now lives in KyaniteLabs/unstuck-coach-protocol
  • Innerscape remains the local/open-source merge target for offline Unstuck

No deployment was performed.


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Chores
    • Removed Unstuck Coach project files, documentation, and build infrastructure from the repository
    • Consolidated Unstuck Coach into a separate dedicated product repository
    • Updated root test suite to reflect project separation

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 7a59b437-c31d-43f0-84ed-55769a04349f

📥 Commits

Reviewing files that changed from the base of the PR and between 186da87 and 1f9f047.

⛔ Files ignored due to path filters (3)
  • EF-COACH/landing/assets/unstuck-admin-bridge.jpg is excluded by !**/*.jpg
  • EF-COACH/landing/assets/unstuck-coach-logo.png is excluded by !**/*.png
  • EF-COACH/landing/assets/unstuck-handoff-card.svg is excluded by !**/*.svg
📒 Files selected for processing (95)
  • EF-COACH/.github/workflows/ci.yml
  • EF-COACH/.gitignore
  • EF-COACH/AGENTS.md
  • EF-COACH/LICENSE
  • EF-COACH/README.md
  • EF-COACH/coach/FIRST_REPLY_SCORECARD.md
  • EF-COACH/coach/FIRST_RUN.md
  • EF-COACH/coach/HANDOFF_CARD.md
  • EF-COACH/coach/PROJECT_INSTRUCTIONS.md
  • EF-COACH/coach/START_HERE.md
  • EF-COACH/coach/examples.md
  • EF-COACH/coach/identity.md
  • EF-COACH/coach/rules.md
  • EF-COACH/demo/before-after.md
  • EF-COACH/demo/transcript-pack.md
  • EF-COACH/demo/whole-person-tour.md
  • EF-COACH/docs/judging/COMPETITION_RULES_TRACE.md
  • EF-COACH/docs/judging/ICM_TRACE.md
  • EF-COACH/docs/judging/JUDGE_BRIEF.md
  • EF-COACH/docs/judging/JUDGE_FAQ.md
  • EF-COACH/docs/judging/JUDGE_SCORECARD.md
  • EF-COACH/docs/judging/PRODUCT_THESIS.md
  • EF-COACH/docs/judging/PUBLICATION_CHECKLIST.md
  • EF-COACH/docs/judging/SUBMISSION.md
  • EF-COACH/docs/judging/WALKTHROUGH.md
  • EF-COACH/docs/judging/WRITEUP.md
  • EF-COACH/docs/judging/judge-walkthrough.md
  • EF-COACH/evals/red-face-tests.md
  • EF-COACH/evals/research-to-behavior-checklist.md
  • EF-COACH/landing/app.js
  • EF-COACH/landing/index.html
  • EF-COACH/landing/posthog.js
  • EF-COACH/landing/styles.css
  • EF-COACH/live-demo/Dockerfile
  • EF-COACH/live-demo/deploy/docker-compose.vps.yml
  • EF-COACH/live-demo/lib/context.mjs
  • EF-COACH/live-demo/lib/llm-client.mjs
  • EF-COACH/live-demo/public/app.js
  • EF-COACH/live-demo/public/index.html
  • EF-COACH/live-demo/public/posthog.js
  • EF-COACH/live-demo/public/styles.css
  • EF-COACH/live-demo/scripts/build-hostinger-compose.mjs
  • EF-COACH/live-demo/scripts/usage-summary.mjs
  • EF-COACH/live-demo/server.mjs
  • EF-COACH/live-demo/test/chat-ui.test.mjs
  • EF-COACH/live-demo/test/compose.test.mjs
  • EF-COACH/live-demo/test/context.test.mjs
  • EF-COACH/live-demo/test/llm-client.test.mjs
  • EF-COACH/live-demo/test/server.test.mjs
  • EF-COACH/llms.txt
  • EF-COACH/package.json
  • EF-COACH/reference/admin-ops-playbooks.md
  • EF-COACH/reference/coaching-protocols.md
  • EF-COACH/reference/mode-router.md
  • EF-COACH/reference/safety-boundaries.md
  • EF-COACH/reference/signal-map.md
  • EF-COACH/reference/source-notes.md
  • EF-COACH/robots.txt
  • EF-COACH/scripts/build-public-bundle.mjs
  • EF-COACH/scripts/final-review-smoke.mjs
  • EF-COACH/scripts/judge-quick-proof.mjs
  • EF-COACH/scripts/prepare-publication-link.mjs
  • EF-COACH/scripts/public-bundle-files.mjs
  • EF-COACH/scripts/render-review-screenshots.mjs
  • EF-COACH/scripts/stage-public-repo.mjs
  • EF-COACH/scripts/verify-admin-ops-playbooks.mjs
  • EF-COACH/scripts/verify-clean-public-stage.mjs
  • EF-COACH/scripts/verify-competition-rules-trace.mjs
  • EF-COACH/scripts/verify-console-behavior.mjs
  • EF-COACH/scripts/verify-eval-coverage.mjs
  • EF-COACH/scripts/verify-final-privacy-scan.mjs
  • EF-COACH/scripts/verify-first-reply-acceptance.mjs
  • EF-COACH/scripts/verify-first-reply-scorecard.mjs
  • EF-COACH/scripts/verify-first-run.mjs
  • EF-COACH/scripts/verify-github-public-url.mjs
  • EF-COACH/scripts/verify-icm-trace.mjs
  • EF-COACH/scripts/verify-judge-brief.mjs
  • EF-COACH/scripts/verify-judge-faq.mjs
  • EF-COACH/scripts/verify-judge-scorecard.mjs
  • EF-COACH/scripts/verify-landing-accessibility.mjs
  • EF-COACH/scripts/verify-landing-copy.mjs
  • EF-COACH/scripts/verify-mode-router.mjs
  • EF-COACH/scripts/verify-product-thesis.mjs
  • EF-COACH/scripts/verify-public-bundle.mjs
  • EF-COACH/scripts/verify-publication-ready.mjs
  • EF-COACH/scripts/verify-source-notes.mjs
  • EF-COACH/scripts/verify-start-here.mjs
  • EF-COACH/scripts/verify-submission-copy.mjs
  • EF-COACH/scripts/verify-submission-surfaces.mjs
  • EF-COACH/scripts/verify-transcript-pack.mjs
  • EF-COACH/scripts/verify-whole-person-tour.mjs
  • EF-COACH/sitemap.xml
  • docs/project-separation.md
  • package.json
  • tests/root-test-surface.test.js
💤 Files with no reviewable changes (92)
  • EF-COACH/.gitignore
  • EF-COACH/docs/judging/PRODUCT_THESIS.md
  • EF-COACH/evals/research-to-behavior-checklist.md
  • EF-COACH/reference/source-notes.md
  • EF-COACH/docs/judging/PUBLICATION_CHECKLIST.md
  • EF-COACH/docs/judging/SUBMISSION.md
  • EF-COACH/docs/judging/WRITEUP.md
  • EF-COACH/reference/mode-router.md
  • EF-COACH/LICENSE
  • EF-COACH/coach/identity.md
  • EF-COACH/reference/admin-ops-playbooks.md
  • EF-COACH/docs/judging/judge-walkthrough.md
  • EF-COACH/evals/red-face-tests.md
  • EF-COACH/sitemap.xml
  • EF-COACH/docs/judging/ICM_TRACE.md
  • EF-COACH/reference/signal-map.md
  • EF-COACH/coach/PROJECT_INSTRUCTIONS.md
  • EF-COACH/coach/FIRST_REPLY_SCORECARD.md
  • EF-COACH/AGENTS.md
  • EF-COACH/scripts/verify-competition-rules-trace.mjs
  • EF-COACH/llms.txt
  • EF-COACH/live-demo/test/llm-client.test.mjs
  • EF-COACH/coach/HANDOFF_CARD.md
  • EF-COACH/scripts/verify-submission-surfaces.mjs
  • EF-COACH/reference/safety-boundaries.md
  • EF-COACH/docs/judging/JUDGE_SCORECARD.md
  • EF-COACH/landing/index.html
  • EF-COACH/scripts/stage-public-repo.mjs
  • EF-COACH/live-demo/deploy/docker-compose.vps.yml
  • EF-COACH/coach/rules.md
  • EF-COACH/scripts/final-review-smoke.mjs
  • EF-COACH/live-demo/test/context.test.mjs
  • EF-COACH/scripts/verify-whole-person-tour.mjs
  • EF-COACH/scripts/public-bundle-files.mjs
  • EF-COACH/docs/judging/WALKTHROUGH.md
  • EF-COACH/scripts/build-public-bundle.mjs
  • EF-COACH/package.json
  • EF-COACH/docs/judging/JUDGE_BRIEF.md
  • EF-COACH/coach/FIRST_RUN.md
  • EF-COACH/scripts/verify-public-bundle.mjs
  • EF-COACH/demo/before-after.md
  • EF-COACH/scripts/verify-admin-ops-playbooks.mjs
  • EF-COACH/.github/workflows/ci.yml
  • EF-COACH/scripts/prepare-publication-link.mjs
  • EF-COACH/scripts/verify-clean-public-stage.mjs
  • EF-COACH/landing/app.js
  • EF-COACH/scripts/verify-first-reply-scorecard.mjs
  • EF-COACH/live-demo/public/posthog.js
  • EF-COACH/scripts/verify-product-thesis.mjs
  • EF-COACH/docs/judging/COMPETITION_RULES_TRACE.md
  • EF-COACH/scripts/verify-landing-accessibility.mjs
  • EF-COACH/scripts/verify-judge-brief.mjs
  • EF-COACH/scripts/verify-start-here.mjs
  • EF-COACH/scripts/verify-source-notes.mjs
  • EF-COACH/live-demo/scripts/usage-summary.mjs
  • EF-COACH/scripts/verify-first-reply-acceptance.mjs
  • EF-COACH/landing/posthog.js
  • EF-COACH/scripts/verify-submission-copy.mjs
  • EF-COACH/live-demo/public/styles.css
  • EF-COACH/live-demo/test/compose.test.mjs
  • EF-COACH/scripts/verify-publication-ready.mjs
  • EF-COACH/live-demo/lib/llm-client.mjs
  • EF-COACH/robots.txt
  • EF-COACH/docs/judging/JUDGE_FAQ.md
  • EF-COACH/reference/coaching-protocols.md
  • EF-COACH/live-demo/lib/context.mjs
  • EF-COACH/live-demo/test/chat-ui.test.mjs
  • EF-COACH/scripts/judge-quick-proof.mjs
  • EF-COACH/demo/whole-person-tour.md
  • EF-COACH/scripts/verify-mode-router.mjs
  • EF-COACH/coach/START_HERE.md
  • EF-COACH/scripts/verify-first-run.mjs
  • EF-COACH/demo/transcript-pack.md
  • EF-COACH/scripts/verify-judge-faq.mjs
  • EF-COACH/coach/examples.md
  • EF-COACH/scripts/verify-final-privacy-scan.mjs
  • EF-COACH/live-demo/test/server.test.mjs
  • EF-COACH/live-demo/public/index.html
  • EF-COACH/scripts/render-review-screenshots.mjs
  • EF-COACH/live-demo/Dockerfile
  • EF-COACH/landing/styles.css
  • EF-COACH/scripts/verify-landing-copy.mjs
  • EF-COACH/README.md
  • EF-COACH/scripts/verify-console-behavior.mjs
  • EF-COACH/live-demo/public/app.js
  • EF-COACH/scripts/verify-icm-trace.mjs
  • EF-COACH/scripts/verify-eval-coverage.mjs
  • EF-COACH/scripts/verify-judge-scorecard.mjs
  • EF-COACH/scripts/verify-github-public-url.mjs
  • EF-COACH/live-demo/scripts/build-hostinger-compose.mjs
  • EF-COACH/live-demo/server.mjs
  • EF-COACH/scripts/verify-transcript-pack.mjs

📝 Walkthrough

Walkthrough

This PR extracts the Unstuck Coach (EF-COACH) project into a dedicated repository and removes it from this codebase. The root-level package configuration and tests are updated to enforce that separation, while a new documentation file explains the change.

Changes

Project Extraction and Separation

Layer / File(s) Summary
Separation documentation
docs/project-separation.md
A new file documents that Dev Learning Archaeologist and Unstuck Coach are separate products, with the EF-COACH subtree now in a dedicated live-product repository. Future work for landing, live-demo, coach contracts, and deployment remains in that repository.
Root test and configuration enforcement
package.json, tests/root-test-surface.test.js
The npm test command is scoped to the root tests/*.test.js only (no longer invokes nested EF-COACH suite), and the root test surface assertion is updated to verify the EF-COACH directory is absent and the test scope is correct.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • KyaniteLabs/dev-learning-archaeologist#65: Both PRs touch EF-COACH/landing/app.js by removing/reworking the same DOM wiring logic (document-root "js" class, .nav-toggle/aria-expanded nav-open handling, and [data-sample] populating #stuck-sentence).
  • KyaniteLabs/dev-learning-archaeologist#61: The main PR removes the existing PostHog client/instrumentation modules (e.g., EF-COACH/landing/posthog.js and EF-COACH/live-demo/public/posthog.js) that the retrieved PR modifies to remove committed PostHog keys and gate tracking on runtime config.
  • KyaniteLabs/dev-learning-archaeologist#54: Both PRs modify the same GitHub Actions workflow (EF-COACH/.github/workflows/ci.yml)—one adds the CI/private-ready-gate job while the main PR removes that workflow entirely.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing the accidental Unstuck Coach (EF-COACH) subproject from the repository.
Description check ✅ Passed The description is mostly complete with summary, verification steps, and repo split details, but does not follow the required template sections for 'Empower Orchestrator checklist' items.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/remove-ef-coach-subproject

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@simongonzalezdc simongonzalezdc merged commit 7a95086 into main Jun 3, 2026
3 checks passed
@simongonzalezdc simongonzalezdc deleted the codex/remove-ef-coach-subproject branch June 3, 2026 07:18
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