Skip to content

docs(architecture): update monorepo.md api/v1 layout and scripts list - #1183

Closed
ibochielizabeth-spec wants to merge 1 commit into
drydocs:mainfrom
ibochielizabeth-spec:docs/monorepo-layout-update
Closed

ibochielizabeth-spec wants to merge 1 commit into
drydocs:mainfrom
ibochielizabeth-spec:docs/monorepo-layout-update

Conversation

@ibochielizabeth-spec

Copy link
Copy Markdown
Contributor

Closes #1146

Summary

Updates apps/docs/architecture/monorepo.md to reflect the current consolidated api/v1 tree layout and complete scripts/ directory entries.

Changes

  • Updated the api/v1 tree in apps/docs/architecture/monorepo.md:
    • Added admin/[resource].ts (POST /api/v1/admin/:resource).
    • Added keepers/[action].ts (POST /api/v1/keepers/:action).
    • Consolidated tx/* into tx/[action].ts (POST /api/v1/tx/:action).
    • Added nested positions/[publicKey]/history.ts (GET /api/v1/positions/:publicKey/history) alongside positions/[publicKey].ts.
  • Updated the scripts/ tree description to list all scripts and directories:
    • deploy-testnet.sh, deploy-mainnet.sh, redeploy-blend-adapter.sh, deploy-external-pinger.sh, build-vercel.sh, verify-contract-addresses.ts, lib/, and load-test/.

Verification

  • Ran pnpm prettier apps/docs/architecture/monorepo.md --check (passes).

@vercel

vercel Bot commented Oct 19, 2026

Copy link
Copy Markdown

@ibochielizabeth-spec is attempting to deploy a commit to the Collins' projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Oct 19, 2026

Copy link
Copy Markdown

@ibochielizabeth-spec Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@collinsezedike collinsezedike left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The api layout and the scripts list match the repo. One line needs a correction, noted inline.

│ ├── _lib/
│ │ └── middleware.ts # CORS + rate-limit helpers shared by handlers
│ └── v1/
│ ├── admin/ # POST /api/v1/admin/:resource

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This handler is GET only: api/v1/admin/[resource].ts rejects any non-GET request with Allow: GET. This comment should read GET /api/v1/admin/:resource.

@collinsezedike

Copy link
Copy Markdown
Collaborator

@ibochielizabeth-spec closing in favour of #1218, which is the same change for #1146 on a current base. The admin GET correction from the inline review is already reflected there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] monorepo.md lists the pre-consolidation api layout and a partial scripts list

2 participants