This repository was archived by the owner on May 9, 2026. It is now read-only.
Fix/encryption#8
Merged
Merged
Conversation
- Introduced comprehensive live integration tests for the Gmail and Notion skills, covering the full encrypted OAuth flow and self-hosted authentication modes. - Implemented new functions for handling authentication, including `authComplete` and `oauthComplete`, to facilitate the OAuth process. - Added test cases to validate skill startup, authentication, and tool functionality, ensuring robust integration with the respective APIs. - Enhanced the test harness to support environment variable configuration for both self-hosted and managed OAuth modes.
- Simplified the encrypted OAuth flow in the test files for Gmail and Notion by removing the separate fetch step for the client key, which is now directly returned in the OAuth callback URL. - Updated environment variable requirements in the tests to reflect the new client key handling. - Adjusted test cases to ensure proper injection of the client key during OAuth credential injection, enhancing clarity and maintainability of the tests.
- Updated the openhuman submodule to the latest commit, ensuring compatibility with recent changes and improvements in the core project.
- Added `ts-node` as a dependency in `package.json` for improved TypeScript execution. - Updated `yarn.lock` to reflect new dependencies and their versions, including `@cspotcode/source-map-support`, `@tsconfig/node10`, `@tsconfig/node12`, `@tsconfig/node14`, and `@tsconfig/node16`. - Refined the live integration test scripts for Gmail and Notion, improving authentication prompts and error handling. - Enhanced the overall structure and readability of the test scripts, ensuring better maintainability and clarity.
- Updated test files for Gmail and Notion to automatically read environment variables from the .env file, simplifying usage. - Removed explicit JWT_TOKEN and BACKEND_URL parameters from the usage instructions, streamlining the command execution process. - Improved overall clarity and maintainability of the test scripts.
…nment management - Updated .env.example to include new environment variables for Gmail and Notion integration, clarifying usage for both OAuth and self-hosted modes. - Enhanced .gitignore to exclude the .fastembed_cache directory, ensuring cleaner project management. - Added a new script (dev-runtime.mjs) for starting the skills runtime with environment variables loaded, streamlining development workflow. - Updated package.json to include a new script for running the development runtime, improving ease of use for developers.
- Updated the Gmail sync logic to utilize the new `isGmailConnected` function for better connection checks. - Enhanced the `gmailFetch` function to differentiate between direct API calls and OAuth proxy usage, improving error handling and token management. - Refined environment variable handling in test scripts, allowing for more streamlined authentication processes. - Improved logging for better debugging and clarity during API interactions.
- Implemented parallel processing for syncing messages and fetching metadata to improve performance and reduce timeouts. - Updated the `gmailFetch` function to handle endpoint paths more efficiently, enhancing API interaction. - Refined error handling in the `callToolSafe` function to provide clearer error messages and improve robustness.
- Rearranged import statements in the Gmail live test script for better organization. - Enhanced error handling in the `callToolSafe` function to improve clarity in error responses. - Streamlined prompts for user input in the main function, improving readability and maintainability. - Consolidated logging statements in the `gmailFetch` function for cleaner output.
|
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 8 minutes and 36 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: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (13)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Skill Submission
Skill name:
skill-name-hereType: [ ] Prompt-only | [ ] Coded (skill.ts)
Description
Brief description of what this skill does and why it's useful.
Checklist
SKILL.mdhas valid YAML frontmatter (name,description)eval(),Function(), or dynamic code executionctx.readData/ctx.writeData)skill.tshas name, description, version{ content: string }npx tsx harness/runner.ts ../skills/my-skillnpm run validatepasses indev/Testing
Describe how you tested this skill:
Category