Strip back to the harness-only skeleton (ready to run through) - #4
Merged
Conversation
A previous build had been committed (scaffolded app, generated services, .power metadata, env config). The demo creates all of that live, so the starting state must be harness-only. Removed (regenerated live during a run): - the scaffolded Vite app: src/, package.json, package-lock.json, tsconfig*, vite.config.ts, eslint.config.js, index.html, public/ - the SDK output: src/generated/ and .power/ - power.config.json (environment-specific) Environment hygiene: - untrack .env (carried an environment URL + id); add .env.example and gitignore .env + power.config.json + .power/ so a run-through never dirties the skeleton. Kept (the harness): AGENTS.md, specs/, dataverse/ skill inputs, the mirrored skills, DEMO-PLAN.md. README rewritten to describe the skeleton, how to run, and how to reset for the next run. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Resets the repository back to a harness-only skeleton intended for a live run-through, removing the previously committed scaffolded Code App and generated Dataverse artifacts so the demo can recreate them live from the specs.
Changes:
- Removed the scaffolded Vite + React + TypeScript app and build/tooling files.
- Removed generated Power Apps/Dataverse artifacts (
src/generated/,.power/,power.config.json). - Added environment hygiene (
.env.example, gitignore updates) and rewroteREADME.mdto describe the skeleton workflow.
Reviewed changes
Copilot reviewed 17 out of 28 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.ts | Removed Vite config as part of stripping the scaffolded app. |
| tsconfig.node.json | Removed Node TS config from the scaffolded app. |
| tsconfig.json | Removed TS project references from the scaffolded app. |
| tsconfig.app.json | Removed app TS config from the scaffolded app. |
| src/main.tsx | Removed React entrypoint from the scaffolded app. |
| src/index.css | Removed default Vite template styles. |
| src/App.tsx | Removed default Vite template app component. |
| src/App.css | Removed default Vite template component styles. |
| src/assets/react.svg | Removed Vite template asset. |
| public/vite.svg | Removed Vite template asset. |
| index.html | Removed Vite HTML entrypoint. |
| package.json | Removed Node project manifest/scripts/deps from the scaffolded app. |
| eslint.config.js | Removed ESLint flat config that belonged to the scaffolded app. |
| power.config.json | Removed environment-specific Power Apps config from the repo. |
| src/generated/services/Eppc_sessionsService.ts | Removed generated Dataverse service output. |
| src/generated/services/Eppc_feedbacksService.ts | Removed generated Dataverse service output. |
| src/generated/models/Eppc_sessionsModel.ts | Removed generated Dataverse model output. |
| src/generated/models/Eppc_feedbacksModel.ts | Removed generated Dataverse model output. |
| src/generated/models/CommonModels.ts | Removed generated common model types. |
| src/generated/index.ts | Removed generated barrel export. |
| .power/schemas/dataverse/sessions.Schema.json | Removed generated .power/ Dataverse schema artifact. |
| .power/schemas/dataverse/feedback.Schema.json | Removed generated .power/ Dataverse schema artifact. |
| .power/schemas/appschemas/dataSourcesInfo.ts | Removed generated .power/ data source info. |
| .gitignore | Ignored per-run .env and generated Power Apps metadata to keep the skeleton clean. |
| .env.example | Added example environment variables file for per-run configuration. |
| .env | Removed committed per-run environment variables file. |
| README.md | Rewritten to describe harness contents, run instructions, and reset workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+32
to
+34
| ```bash | ||
| git reset --hard <skeleton-commit> && git clean -fdx | ||
| ``` |
Comment on lines
+13
to
+14
| - **`dataverse/`** — the schema (`eppc-schema.yaml`) and seed (`eppc-sessions.yaml`) the Dataverse | ||
| skills execute against, compiled from the specs. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resets the repo to its starting state for a live run-through: harness only, no Code App, no Dataverse artifacts. A previous build had been committed; the demo is supposed to create all of that live, so it must not be in the starting commit.
Removed (all regenerated live during a run)
src/,package.json,package-lock.json,tsconfig*,vite.config.ts,eslint.config.js,index.html,public/src/generated/(typed services) and.power/(data-source metadata)power.config.json(environment-specific)Environment hygiene
.env(it carried an environment URL + ID, no tokens). Added.env.exampleand gitignored.env,power.config.json, and.power/so a run-through never dirties the skeleton.Kept (the harness)
AGENTS.md,specs/, thedataverse/skill inputs, the mirrored.agents/.claudeskills,DEMO-PLAN.md. README rewritten to describe the skeleton, how to run, and how to reset for the next run.Two things to confirm
dataverse/eppc-sessions-seed.yamllooks like a stale duplicate ofdataverse/eppc-sessions.yaml(no header comment, different indentation, fewer lines). I kept it rather than delete data on a guess. Ifeppc-sessions.yamlis the canonical seed input (its header says it is), deleteeppc-sessions-seed.yamlso the agent can't feed the wrong one.dataverse/*.yamlinputs are what the skills actually execute, compiled from the specs. Verify they still matchspecs/dataverse-schema.md(§6 choices) andspecs/seed-sessions.md(the 20 sessions) on the dry run, or the demo builds the wrong schema/data.Copilot wiring (
.github/) comes in a separate PR.🤖 Generated with Claude Code