refactor: flatten role folders into single .ts files (develop-sense & develop-workflow) #13

New Issue

xiaoju · 2026-04-29T12:19:05Z

xiaoju commented

2026-04-29 12:19:05 +00:00

What

Each role under develop-sense/roles/ and develop-workflow/roles/ currently has a folder with index.ts + prompt.ts. The index.ts files are trivially short (6 lines, just schema + type). Merge them into single .ts files and remove the folders.

Current Structure

workflows/develop-sense/roles/
  planner/
    index.ts    # 6 lines: schema + type only
    prompt.ts   # prompt function
  coder/
    index.ts
    prompt.ts
  reviewer/
    index.ts
    prompt.ts
  tester/
    index.ts
    prompt.ts
  committer/
    index.ts    # re-export from _shared

Same structure for develop-workflow/roles/.

Target Structure

workflows/develop-sense/roles/
  planner.ts    # schema + type + prompt function, all in one
  coder.ts
  reviewer.ts
  tester.ts
  committer.ts  # re-export from _shared (stays simple)

Same for develop-workflow/roles/.

Rules

For each role folder (planner, coder, reviewer, tester): merge index.ts and prompt.ts into a single <role>.ts file, then delete the folder.
For committer/: it only has index.ts (re-export from _shared), just move it to committer.ts and delete the folder.
Update all imports in build.ts (and anywhere else that imports from these roles) — change ./roles/planner/index.js → ./roles/planner.js etc.
Keep all exports identical (schema, type, prompt function) — no API changes.
Run pnpm build at repo root to verify.

Scope

workflows/develop-sense/roles/ — 5 roles
workflows/develop-workflow/roles/ — 5 roles
workflows/develop-sense/build.ts — update imports
workflows/develop-workflow/build.ts — update imports

## What Each role under `develop-sense/roles/` and `develop-workflow/roles/` currently has a folder with `index.ts` + `prompt.ts`. The `index.ts` files are trivially short (6 lines, just schema + type). Merge them into single `.ts` files and remove the folders. ## Current Structure ``` workflows/develop-sense/roles/ planner/ index.ts # 6 lines: schema + type only prompt.ts # prompt function coder/ index.ts prompt.ts reviewer/ index.ts prompt.ts tester/ index.ts prompt.ts committer/ index.ts # re-export from _shared ``` Same structure for `develop-workflow/roles/`. ## Target Structure ``` workflows/develop-sense/roles/ planner.ts # schema + type + prompt function, all in one coder.ts reviewer.ts tester.ts committer.ts # re-export from _shared (stays simple) ``` Same for `develop-workflow/roles/`. ## Rules 1. For each role folder (planner, coder, reviewer, tester): merge `index.ts` and `prompt.ts` into a single `<role>.ts` file, then delete the folder. 2. For `committer/`: it only has `index.ts` (re-export from `_shared`), just move it to `committer.ts` and delete the folder. 3. Update all imports in `build.ts` (and anywhere else that imports from these roles) — change `./roles/planner/index.js` → `./roles/planner.js` etc. 4. Keep all exports identical (schema, type, prompt function) — no API changes. 5. Run `pnpm build` at repo root to verify. ## Scope - `workflows/develop-sense/roles/` — 5 roles - `workflows/develop-workflow/roles/` — 5 roles - `workflows/develop-sense/build.ts` — update imports - `workflows/develop-workflow/build.ts` — update imports

xiaoju referenced this issue from a commit

2026-04-29 12:21:46 +00:00

refactor: flatten role folders into single .ts files

xiaoju referenced a pull request that will close this issue

2026-04-29 12:21:55 +00:00

refactor: flatten role folders into single .ts files #14

xiaoju closed this issue

2026-04-29 12:23:34 +00:00

Sign in to join this conversation.