chore: CI, issue/PR templates, package.json metadata, README windows note#124
Closed
chore: CI, issue/PR templates, package.json metadata, README windows note#124
Conversation
…windows note
- .github/workflows/ci.yml: typecheck + lint + build on PR/push
- .github/ISSUE_TEMPLATE/{bug_report,feature_request,model_routing,config}.yml + PR template
- package.json: name (cadam), version (0.1.0), description, homepage, repository, bugs, author, license, keywords
- README.md: flag Docker Desktop requirement on Windows (closes #118), correct React 19
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
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
1 participant
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.
Summary
Repo hygiene pass: adds CI, issue/PR templates, fixes
package.jsonmetadata, and documents the Docker Desktop requirement on Windows.This is the first PR in an unattended ops loop Eve is running for CADAM. All Eve-authored branches are prefixed
eve/...so they're easy to filter and revert. No code changes touch product surface.Why
The repo is at 2.6k stars with no automated guardrails on PRs (no GitHub Actions workflows defined),
package.jsonstill ships the placeholdervite-react-typescript-starter/0.0.0, and Windows users keep hitting the same wall onnpx supabase start.Closes #118.
Changes
.github/workflows/ci.yml: typecheck + lint + build on every PR and push tomaster. Concurrency-cancels in-flight runs on the same ref..github/ISSUE_TEMPLATE/{bug_report,feature_request,model_routing,config}.yml: structured templates so we get reproducible bug reports (model + prompt + browser) and contributor-friendly feature requests. Disables blank issues; surfaces Discord and the live demo as contact links..github/PULL_REQUEST_TEMPLATE.md: standard summary / why / changes / testing / risk template.package.json:name->cadam,version->0.1.0(matches the v0.1.0 GitHub release), plusdescription,homepage,repository,bugs,author,license(GPL-3.0-or-later),keywords. No dependency or script changes.README.md: adds Docker Desktop to Prerequisites with a Windows-specific note (closes npx supabase start -> error #118), corrects the "Built With" line from React 18 to React 19 (matches the actualreact@19.1.0dependency).Testing
npm run typecheck,npm run lint,npm run buildscripts (no script changes needed).VITE_*env values in CI don't break the build (Vite usually treats undefinedimport.meta.envasundefinedat build time; the placeholders are belt-and-braces).npm cisucceeds againstpackage-lock.jsonon Node 20.Risk
Low.
package.jsonrename changes thenamefield, which could affect any downstream tool that keys off it (none known in this repo).GPL-3.0toGPL-3.0-or-laterto match the actualLICENSEtext and the SPDX recommendation. If you'd rather keep it pinned toGPL-3.0, happy to revert that single change.Follow-up PRs already queued
eve/vitest-harness- Vitest setup + 4 smoke tests (mutation hooks, message service, OpenSCAD worker happy path, parameter parser).eve/regression-tests-115-113- regression tests targeting the abort behavior in fix(chat): abort streaming fetch on unmount / stopGenerating #115 and the pending-tool-call recovery in fix(parametric-chat): prevent forever-pending tool calls after edge kill #113, then re-request review on those PRs.eve/readme-demo-gif- replace static screenshots with a 3s loop on the GitHub landing surface.Drafted and pushed by Eve operating on Zach's full-approval grant. PR is opened as a draft - flip to ready when you've eyeballed it.
Summary by cubic
Adds CI and repo templates, fixes
package.jsonmetadata, and documents the Windows Docker requirement. No product behavior changes.CIworkflow to typecheck, lint, and build on PRs and pushes tomaster; cancels in‑flight runs; uses Node 20 with safeVITE_*placeholders.package.json:name→cadam,version→0.1.0, plus description, homepage, repository, bugs, author, licenseGPL-3.0-or-later, keywords (no deps/scripts changed).README.md: note that Windows needs Docker Desktop fornpx supabase startand corrected React 19. Closes npx supabase start -> error #118.Written for commit 3a4b2f4. Summary will update on new commits.