Release 20251210 #1798
Merged
Release 20251210 #1798
Conversation
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
Fix/plus pricing
#1789) feat: Add `endpoint` configuration and use it for public file content URLs in the skill engine.
* Enhance ToolsDependencyChecker to refetch user tools on settings modal close * feat: integrate useSiderStoreShallow to manage settings modal state * fix: implement refetching of user tools when the settings modal is closed, ensuring updated installation status * refactor: optimize user tools data handling with nullish coalescing and optional chaining for safer property access * Refactor ToolsDependencyChecker to utilize refetchUserTools for improved user tools data handling * refactor: replace queryClient.refetchQueries with refetchUserTools for better clarity and performance * refactor: ensure proper dependency array in useEffect to prevent potential infinite loops * refactor: maintain compliance with optional chaining and nullish coalescing for safer property access * refactor: remove unused queryClient import in ToolsDependencyChecker for improved clarity * refactor: eliminate unnecessary useQueryClient import to streamline component logic * refactor: maintain compliance with optional chaining and nullish coalescing for safer property access
* feat(tracer): decouple tracer and logger configuration - Refactor initTracer() to read config from environment variables directly - Remove pino-loki dependency, use JSON stdout for k8s log collection - Add traceId injection via mixin for Loki → Tempo correlation - Simplify Langfuse enablement: enabled when LANGFUSE_BASE_URL is set - Remove LANGFUSE_ENABLED and LOKI_HOST environment variables 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]> * chore: update pnpm-lock.yaml 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]> --------- Co-authored-by: HanYuanxi <[email protected]> Co-authored-by: Claude Opus 4.5 <[email protected]>
* refactor: update NodeRenderer component to enhance file download handling - Removed the deprecated downloadNodeData function and integrated useDownloadFile hook for improved file download functionality. - Updated handleDownload logic to utilize the new handleDownload method, ensuring proper file type handling. - Ensured compliance with coding standards, including optional chaining and nullish coalescing for safer property access. * fix: update file name handling in NodeRenderer component - Set the file name to the node title within the handleDownloadNode function to ensure accurate file naming during downloads. - Maintained compliance with coding standards, including the use of optional chaining and nullish coalescing for safer property access.
#1791) chore: update deployment workflows to check for existing Docker images in ECR - Added a step to check if the Docker image exists in ECR before building and pushing, optimizing the deployment process. - Moved the Docker Buildx setup step to only execute if the image does not exist, enhancing efficiency. - Ensured compliance with existing coding standards and best practices.
* feat(action): use safeParseJSON for input and output parsing * feat(action): enhance input and output handling with safeParseJSON defaults
feat(skill): increase concurrency for skill invoker - Enhanced the SkillProcessor by setting the concurrency option using SKILL_DEFAULTS.INVOKE_WORKER_CONCURRENCY for improved performance. - Ensured compliance with coding standards, including the use of optional chaining and nullish coalescing for safer property access.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ 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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Tip
Close issue syntax:
Fixes #<issue number>orResolves #<issue number>, see documentation for more details.Impact Areas
Please check the areas this PR affects:
Screenshots/Videos
Checklist
Important
Please review the checklist below before submitting your pull request.
dev/reformat(backend) andcd web && npx lint-staged(frontend) to appease the lint gods