Updated Mastra Packages, and updated .gitignore to remove files that will cause noise in workspace.#147
Conversation
- deleted CONTINUITY_ses_456e.md: Removed session summary for tool improvements in Mastra, including goals, progress, and next steps. - deleted CONTINUITY_ses_4571.md: Removed session summary for project analysis of AgentStack, detailing goals, progress, and key decisions. - deleted CONTINUITY_ses_458c.md: Removed session summary for fixing TypeScript errors in image-tool.ts and implementing new tools for AgentStack. - deleted CONTINUITY_ses_45a2.md: Removed session summary for migrating tools from OpenTelemetry API to tracingContext pattern. - deleted CONTINUITY_ses_45e3.md: Removed session summary for implementing lifecycle hooks and tracing in e2b.ts. - deleted CONTINUITY_ses_45e6.md: Removed session summary for completing e2b.ts with typed contexts and standardized tracing. - deleted CONTINUITY_ses_45fc.md: Removed session summary for analyzing AgentStack project structure and technology stack. - deleted CONTINUITY_ses_4603.md: Removed session log for reading web-scraper-tool.ts for reference. - deleted CONTINUITY_ses_460a.md: Removed session summary for ensuring all tools use lifecycle hooks and request context. - deleted CONTINUITY_ses_460b.md: Removed session summary for adding lifecycle hooks and RequestContext usage to tools. Co-authored-by: Copilot <copilot@github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
Reviewer's GuideUpdates Mastra and related tooling dependencies, adjusts npm scripts to run via dotenvx, and cleans up repo noise by ignoring and deleting continuity ledger markdown files. Flow diagram for continuity ledger cleanup and gitignoreflowchart TD
A["Existing CONTINUITY_ses_*.md files in thoughts/ledgers"] --> B["Update .gitignore to ignore CONTINUITY_ses_*.md"]
B --> C["Delete tracked CONTINUITY_ses_*.md files from repo"]
C --> D["Git history retains removed files"]
D --> E["Future CONTINUITY_ses_*.md files are untracked and ignored"]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 31 minutes and 17 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (19)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
🤖 Hi @ssdeanx, I've received your request, and I'm working on it now! You can track my progress in the logs for more details. |
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (3 files)
Reviewed by grok-code-fast-1:optimized:free · 107,569 tokens |
|
🤖 I'm sorry @ssdeanx, but I was unable to process your request. Please see the logs for more details. |
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the project version to 1.0.47, upgrades numerous dependencies including @mastra and OpenTelemetry, and modifies npm scripts to use dotenvx. It also removes several session summary markdown files and introduces new patterns to the .gitignore file. The review feedback highlights that several of the new ignore patterns, such as *mastra/, *tests/, and *data/, are far too broad and risk excluding essential source code and test files from version control. Additionally, the feedback points out non-standard path separators and redundant entries in the .gitignore file.
| conductor/setup_state.json | ||
| *conductor/ | ||
| *dist/ | ||
| *mastra/ |
There was a problem hiding this comment.
The pattern *mastra/ is overly broad and will ignore the src/mastra/ directory, which contains the core logic of the application. This will prevent source files from being tracked by Git. If the intention was to ignore the .mastra cache/metadata directory, it is already covered on lines 34 and 82.
| tests/test-results/**/*.json | ||
| tests/test-results/.dot | ||
| tests/*.test.ts | ||
| *tests/ |
There was a problem hiding this comment.
The pattern *tests/ (and */tests/ on line 253) will ignore the tests/ directory. Since this project maintains a high test coverage target, the test files should be tracked in the repository.
| *.sessions/ | ||
| *.logs/ | ||
| *.log/ | ||
| *data/ |
There was a problem hiding this comment.
The pattern *data/ will ignore the src/mastra/data/ directory. If this folder contains essential application data or configuration files, it should not be ignored.
| tests/test-results/.dot | ||
| tests/*.test.ts | ||
| *tests/ | ||
| /tests/test-results\test-results.json |
There was a problem hiding this comment.
Git ignore patterns should use forward slashes (/) as path separators, even on Windows. The backslash (\) used here is non-standard. Additionally, this specific file is already covered by the pattern on line 242.
/tests/test-results/test-results.json
There was a problem hiding this comment.
Pull request overview
This PR updates Mastra-related package versions and reduces repository noise by removing generated/session ledger artifacts and expanding .gitignore rules.
Changes:
- Bumped
@mastra/*(and a few other) dependency versions and incremented the package version to1.0.47. - Updated some npm scripts to run
start:*commands viadotenvx. - Deleted multiple
thoughts/ledgers/CONTINUITY_*.mdsession files and added additional.gitignoreentries intended to prevent future workspace noise.
Reviewed changes
Copilot reviewed 18 out of 20 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
package.json |
Updates Mastra/dependency versions and tweaks start scripts to use dotenvx. |
.gitignore |
Adds new ignore rules for test results and various tooling folders (some patterns are overly broad). |
thoughts/ledgers/CONTINUITY_ses_*.md (multiple) |
Removes committed session ledger files from the repo. |
| "@mastra/client-js": "^1.14.0-alpha.10", | ||
| "@mastra/cloudflare": "^1.3.2-alpha.0", | ||
| "@mastra/cloudflare-d1": "^1.0.5-alpha.0", | ||
| "@mastra/convex": "^1.0.8-alpha.0", | ||
| "@mastra/core": "^1.26.0-alpha.8", | ||
| "@mastra/core": "^1.26.0-alpha.10", | ||
| "@mastra/daytona": "^0.2.1", | ||
| "@mastra/deployer": "^1.26.0-alpha.8", | ||
| "@mastra/deployer": "^1.26.0-alpha.10", | ||
| "@mastra/deployer-cloudflare": "^1.1.23", | ||
| "@mastra/docker": "^0.0.0-alpha.0", | ||
| "@mastra/duckdb": "^1.1.2", | ||
| "@mastra/editor": "^0.7.17-alpha.2", | ||
| "@mastra/docker": "^0.1.0-alpha.0", | ||
| "@mastra/duckdb": "^1.1.3-alpha.0", | ||
| "@mastra/editor": "^0.7.17-alpha.3", |
There was a problem hiding this comment.
After bumping @mastra/* packages (e.g. @mastra/core), the lockfile indicates these packages require Node >=22.13.0, but this repo’s engines.node is still >=20.9.0. This mismatch can lead to install/runtime failures depending on the environment; consider updating engines.node (and CI/runtime docs) to match, or pin Mastra dependencies to versions compatible with Node 20.
Summary by Sourcery
Update dependencies and tooling around Mastra and related packages while cleaning up workspace noise files.
Enhancements:
Chores: