Skip to content

chore: reorganize modules into modules/ with MODULES.md index - #180

Open
seanspeaks wants to merge 8 commits into
nextfrom
cursor/reorganize-modules-a709
Open

seanspeaks wants to merge 8 commits into
nextfrom
cursor/reorganize-modules-a709

Conversation

@seanspeaks

@seanspeaks seanspeaks commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Layout

Flat module tree (package names unchanged):

Path Purpose
modules/<name>/ All API modules (was packages/v1-ready/* + packages/needs-updating/*)
archive/qbo/ Retired QBO module (replaced by quickbooks in #111) — not published
tools/scale-test/ Scale-test lambda + OpenAPI (was services/ + openapi/) — private / not published

MODULES.md is generated (npm run modules:index) with columns: module, package name, version, auth type, status (v1-ready / needs-updating / archived).

Rebased onto latest next (post-#103)

Rebased cursor/reorganize-modules-a709 onto origin/next (7d45d6e, includes clio regional OAuth from #103).

  • No brand-new modules landed under packages/* on next.
  • Clio source updates renamed cleanly into modules/clio/.
  • Newly added regions.test.ts was left under packages/v1-ready/clio/test/ by the rebase and was moved with a pure git mv → modules/clio/test/regions.test.ts.
  • Local merge of this branch into next completes with no conflicts.
  • packages/ is gone; npx lerna ls still reports 32 packages under modules/*.

Commits

  1. Pure git mv of modules + archive qbo + scale-test tools
  2. Workspaces / lerna / .autorc / Dependabot / lockfile / scale-test path aliases
  3. MODULES.md generator + CI check + docs
  4. Phase 2 rebase helper + dry-run report (no force-pushes to other PRs)
  5. Post-rebase: relocate clio regions.test.ts into modules/

Release / auto

  • Publishable: modules/* only (32 packages via lerna ls / auto shipit --dry-run).
  • Not published: archive/*, tools/*.

Verification (post-rebase)

$ npm run modules:index:check
MODULES.md is up to date.          # exit 0

$ npm install
up to date, audited 3427 packages  # exit 0

$ npm test -w @friggframework/api-module-clio
✓ test/regions.test.ts (10 tests)  # PASS

$ npm test --foreground-scripts --workspaces --if-present
# ~10 PASS (incl. clio, attio, fathom, fireflies, gong, otter, quo, reevo, scale-test…)
# ~23 FAIL — pre-existing MongoMemoryServer / libcrypto.so.1.1 env gaps (not path regressions)

Phase 2 note

Other open module PRs are not rebased here; that waits until #180 merges.

Open in Web Open in Cursor 

cursoragent and others added 8 commits October 21, 2026 13:12
Pure git mv only (100% renames, no content edits):
- packages/v1-ready/* and packages/needs-updating/* -> modules/<name>/
- packages/needs-updating/qbo -> archive/qbo (replaced by quickbooks)
- services/frigg-scale-test-lambda and openapi/ -> tools/scale-test/

Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
- Point npm workspaces at modules/* and the private scale-test lambda
- Restrict lerna packages to modules/* (exclude archive/* and tools/*)
- Add .autorc aligned with package.json auto npm monorepo settings
- Update Dependabot to the root lockfile; fix frigg-scale-test path aliases
- Regenerate package-lock.json via npm install; add pure-rand for Jest 29

Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
- Add scripts/gen-modules-index.js (module, package, version, auth, status)
- Commit generated MODULES.md and a CI job that fails when it is stale
- Document flat modules/ layout in README and CONTRIBUTING
- Point remaining docs (zoho-crm, scale-test CLAUDE) at new paths

Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
Prepares (does not execute) force-with-lease rebases for open PRs into
next: relocate packages/v1-ready|needs-updating paths to modules/,
regenerate package-lock on conflict, and run per-module tests.

Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
Rebase onto origin/next brought PR #103 (clio regional OAuth). Source
updates renamed cleanly into modules/clio/; the newly added test file
was left under packages/v1-ready/ and is relocated as a pure rename.

Co-authored-by: Sean Matthews <sean.matthews@lefthook.co>
@cursor
cursor Bot force-pushed the cursor/reorganize-modules-a709 branch from 207017a to 7f9ef16 Compare October 21, 2026 13:14
@seanspeaks
seanspeaks marked this pull request as ready for review October 22, 2026 00:36
@seanspeaks seanspeaks added the auth-test pending Code and CI green; frigg auth test waits on credentials label Oct 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auth-test pending Code and CI green; frigg auth test waits on credentials

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants