Skip to content

chore: decouple embedded showcase from Comptextv7 validation#195

Merged
ProfRandom92 merged 6 commits into
mainfrom
cleanup/remove-embedded-showcase
May 21, 2026
Merged

chore: decouple embedded showcase from Comptextv7 validation#195
ProfRandom92 merged 6 commits into
mainfrom
cleanup/remove-embedded-showcase

Conversation

@ProfRandom92
Copy link
Copy Markdown
Owner

Summary

  • Decouples the embedded showcase/app from root validation scripts.
  • Removes showcase install/build/typecheck steps from CI and agent checks.
  • Updates validation and agent workflow documentation to make dashboard/app the only Node application validated in Comptextv7.
  • Documents that the showcase now lives in ProfRandom92/comptext-v7-monaco-showcase.
  • Notes the recovery branch archive/showcase-before-extraction for the previous embedded showcase state.

Safety

  • No dependency upgrades.
  • No dashboard changes.
  • No Python/core/runtime changes.
  • No merge into main.
  • Backup branch exists: archive/showcase-before-extraction.

Follow-up

The embedded showcase files are not deleted in this PR yet. This PR first removes the active coupling from scripts, CI, and docs so CI can validate the dashboard-only flow. Physical removal of showcase/app, showcase/assets, and docs/SHOWCASE_READINESS.md can happen in a follow-up once the full file list is handled safely.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request focuses on decoupling the Monaco showcase application from the Comptextv7 repository, moving it to a separate repository while preserving historical state in an archive branch. The changes involve updating docs/AGENT_WORKFLOW.md, docs/validation.md, and package.json to remove showcase-specific scripts, validation steps, and directory references. Feedback highlights that the removal of specific examples for unsanitized identifiers (like VIN/FIN) weakens the security policy and that the deletion of the detailed experiment report consumption table and the review checklist reduces essential guidance for agents. It is recommended to restore these sections to maintain clear data privacy and quality assurance standards.

Comment thread docs/AGENT_WORKFLOW.md Outdated
Comment thread docs/AGENT_WORKFLOW.md Outdated
Comment thread docs/AGENT_WORKFLOW.md
Copy link
Copy Markdown
Owner Author

Addressed the documentation review feedback.

Restored and expanded the professional agent workflow guidance while keeping the showcase decoupling intact:

  • restored detailed sanitization/security constraints
  • restored benchmark/regression/sanitization/forensic report consumption guidance
  • restored review checklist and repository ownership guidance
  • clarified that Monaco showcase work now belongs in ProfRandom92/comptext-v7-monaco-showcase
  • preserved the recovery branch archive/showcase-before-extraction
  • kept the cleanup limited to documentation, root validation scripts, and CI decoupling only

No dependency changes, runtime changes, or dashboard logic changes were introduced.

@ProfRandom92 ProfRandom92 marked this pull request as ready for review May 21, 2026 18:23
@ProfRandom92 ProfRandom92 merged commit d7d6b51 into main May 21, 2026
7 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.

1 participant