Skip to content

feat(app-builder): prove installed CLI generation - #147

Merged
kattsushi merged 5 commits into
feat/app-builder-v1-parity-migrationfrom
feat/app-builder-public-cli-success-proof
Aug 14, 2026
Merged

feat(app-builder): prove installed CLI generation#147
kattsushi merged 5 commits into
feat/app-builder-v1-parity-migrationfrom
feat/app-builder-public-cli-success-proof

Conversation

@kattsushi

Copy link
Copy Markdown
Contributor

Closes #146

Part of #131. Depends on #143 and its independently reviewed correction merged as #145.

PR Type

  • New feature (type:feature)

Summary

  • prove generation through the packed and installed public node_modules/.bin/effectify-app-builder JSONL CLI
  • prove custom @acme Operations, Task, and AdminConsole naming against the exact four-project generated Nx graph
  • run generated test, typecheck, and build checks, exercise live UUID CRUD, prove byte-identical replay with no second writes, and clean up the successful isolated run

Changes

Area Change
Public CLI Expose and execute the installed effectify-app-builder binary with stdin JSONL generation events.
Generated graph Keep the custom workspace to exactly @acme/admin-console plus the Operations application, domain, and infrastructure projects.
Runtime proof Exercise generated checks and live add, complete, and remove behavior through the generated implementation.
Replay and cleanup Require byte-identical regeneration with no changed or second-write paths, then remove the successful run's registry store and workspace.

Test plan

  • pnpm nx run-many --projects @effectify/app-builder-contracts --targets build,typecheck,test,lint --nxBail --skip-nx-cache: PASS, 24 files / 67 tests; build, typecheck, and lint passed; 6 existing lint warnings and 0 errors.
  • pnpm nx run-many --projects @effectify/app-builder-generation --targets build,typecheck,test,lint --nxBail --skip-nx-cache: PASS, 8 files / 29 tests; build, typecheck, and lint passed.
  • pnpm nx run @effectify/app-builder-nx-plugin:{build,typecheck,test,lint} --skip-nx-cache as four sequential invocations: PASS, 1 file / 9 tests; all targets passed.
  • pnpm nx run @effectify/app-builder-cli:{build,typecheck,test,lint} --skip-nx-cache as four sequential invocations: PASS, 1 file / 9 tests; all targets passed.
  • pnpm nx run @effectify/app-builder-e2e:{typecheck,build,lint} --skip-nx-cache as sequential invocations: PASS, all targets passed.
  • pnpm nx run @effectify/app-builder-e2e:test --skip-nx-cache: PASS, dependent public builds plus 1 file / 1 test; 27.0s Nx run and 24.32s Vitest.
  • pnpm exec oxfmt --check <eight changed paths>: PASS, all matched files correctly formatted; 7 recognized files because .template is not a formatter input.
  • git diff --check: PASS.
  • Independent read-only validation: PASS at tree 30a43782fbbd316e9eb54635a4334841aa8fe660, binary diff SHA-256 68740e732bfb7432cdb56933e980b9cbf9325623e0fe6e669fdc2a1e23067cf1, with no process, workspace, root graph, index, or lockfile leak.

Contributor checklist

  • Linked approved issue Prove successful installed App Builder CLI generation #146.
  • Added exactly one type:* label, type:feature.
  • Tests and runtime evidence are included with the behavior.
  • Used the conventional commit feat(app-builder): prove installed CLI generation.
  • No Co-Authored-By trailers.

Chain Context

Field Value
Chain App Builder reusable generation kernel, Feature Branch Chain
Tracker issue #131
Position Unit 6A
Base feat/app-builder-v1-parity-migration at 2ad43e0cca44a0ed34db27fa29b344a71fc8e214
Depends on #143, including the independently reviewed #145 correction merged into its branch
Follow-up Unit 6B as the next child; Unit 7 afterward
Review budget 398 / 400 changed lines, 167 additions and 231 deletions
Starts at Unit 5 parity migration and merged declarative-template correction from #143/#145
Ends with Successful installed public CLI generation, exact custom Nx graph, generated checks, live CRUD, byte-identical replay, and successful cleanup

Chain Overview

#143 (+ merged #145) → 📍 Unit 6A → Unit 6B → Unit 7 → frozen #130

Scope

Rollback boundary

Revert the single Unit 6A commit to restore exactly the eight changed paths to 2ad43e0cca44a0ed34db27fa29b344a71fc8e214. This removes only the public installed-CLI success proof and exact-four-project Nx correction, without touching #143/#145, Unit 6B, Unit 7, or PR #130.

Autonomy

  • CI is expected to pass for this PR branch.
  • This PR has one deliverable scope.
  • This PR can be rolled back without unrelated changes.
  • Independent tests and runtime evidence cover this unit.

@kattsushi kattsushi added the type:feature New feature work label Aug 13, 2026
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 8a1090fc-cb33-4de3-94a2-3bee3eed51aa

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@kattsushi
kattsushi marked this pull request as ready for review August 13, 2026 17:38
@kattsushi
kattsushi merged commit 8cfeaa0 into feat/app-builder-v1-parity-migration Aug 14, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New feature work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant