Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: organize ts files and tooling #466

Merged
merged 1 commit into from
Feb 25, 2025
Merged

chore: organize ts files and tooling #466

merged 1 commit into from
Feb 25, 2025

Conversation

0xRAG
Copy link
Contributor

@0xRAG 0xRAG commented Feb 25, 2025

What changed?

Moves all TypeScript ecosystem files & tooling into the root typescript/ folder for better organization & separation between TypeScript and Python, and in preparation for the incoming addition of ecosystem-specific changelog management tooling.

In general, we are now treating the coinbase/agentkit repo as a repo that houses two distinct monorepos: one for TypeScript and one for Python.

All relevant readmes and contributing files have been updated, along with GitHub Actions.

@0xRAG 0xRAG requested a review from murrlincoln as a code owner February 25, 2025 16:14
@cb-heimdall
Copy link

cb-heimdall commented Feb 25, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@github-actions github-actions bot added documentation Improvements or additions to documentation example New example agent framework extension New framework extension needs triage labels Feb 25, 2025
@0xRAG 0xRAG force-pushed the chore/organize-ts branch 3 times, most recently from 023bba4 to 59e890f Compare February 25, 2025 16:34
@@ -15,7 +15,8 @@ jobs:
with:
node-version: "18"
registry-url: "https://registry.npmjs.org"
- run: npm ci
Copy link
Contributor Author

Choose a reason for hiding this comment

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

npm ci is ran in the following step

# Install dependencies in parent directory first
- run: npm install
# formatting/lint in working directory
cache-dependency-path: ./typescript
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@CarsonRoscoe CarsonRoscoe left a comment

Choose a reason for hiding this comment

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

Code wise, happy with the refactor. Makes sense.

Readme update wise, your changes are clear and make sense.

Functionality wise, can confirm:

  • Root does nothing, no commands work from root
  • From new root (/typescript), every core turbo command (build, format/format:check, lint/lint:fix) is still being ran on all 8 node packages
  • Build still injects @coinbase/agentkit into the chatbot examples properly for good devx

Overall, LGTM

@0xRAG 0xRAG merged commit b04d31b into main Feb 25, 2025
14 checks passed
@0xRAG 0xRAG deleted the chore/organize-ts branch February 25, 2025 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation example New example agent framework extension New framework extension
Development

Successfully merging this pull request may close these issues.

3 participants