Fix: deep research use case #493
Conversation
🦋 Changeset detectedLatest commit: 452e0b6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Warning Rate limit exceeded@leehuwuj has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 0 minutes and 55 seconds before requesting another review. ⌛ 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. 📒 Files selected for processing (2)
WalkthroughThis pull request introduces modifications to the deep research workflow and related components. The changes primarily focus on enhancing the research planning process by adding a Changes
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (1)
templates/components/agents/python/deep_research/app/workflows/agents.py (1)
67-74: Ensure consistency in prompt formattingFor consistency, consider formatting all
enhanced_promptstrings using f-strings, even when they do not contain placeholders. This makes the code more maintainable and reduces the risk of formatting issues in the future.Apply this diff to use f-strings consistently:
if total_questions == 0: # Avoid writing a report without any research context - enhanced_prompt = """ + enhanced_prompt = f""" The student has no questions to research. Let start by asking some questions. """
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
templates/components/agents/python/deep_research/app/workflows/agents.py(3 hunks)templates/components/agents/python/deep_research/app/workflows/deep_research.py(7 hunks)templates/types/streaming/fastapi/main.py(2 hunks)
🧰 Additional context used
📓 Path-based instructions (3)
templates/types/streaming/fastapi/main.py (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
templates/components/agents/python/deep_research/app/workflows/agents.py (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
templates/components/agents/python/deep_research/app/workflows/deep_research.py (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
⏰ Context from checks skipped due to timeout of 90000ms (45)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --example-file)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --no-files)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --no-files)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --example-file)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --no-files)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --no-files)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --example-file)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --no-files)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --no-files)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --llamacloud)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --example-file)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --no-files)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --llamacloud)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --example-file)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --no-files)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --llamacloud)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --example-file)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --no-files)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --llamacloud)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --example-file)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --no-files)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --no-files)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --llamacloud)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --example-file)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --no-files)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --no-files)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --llamacloud)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --example-file)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --no-files)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --no-files)
templates/components/agents/python/deep_research/app/workflows/deep_research.py
Show resolved
Hide resolved
templates/components/agents/python/deep_research/app/workflows/agents.py
Outdated
Show resolved
Hide resolved
leehuwuj
force-pushed
the
fix/deep-research
branch
from
17b988a to
6bea214
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (6)
templates/types/streaming/nextjs/app/components/ui/accordion.tsx (2)
8-20: Consider exporting TypeScript types for better type safety.The component types could be exported to allow consumers to properly type their props.
+export type AccordionProps = React.ComponentPropsWithoutRef<typeof AccordionPrimitive.Root>; +export type AccordionItemProps = React.ComponentPropsWithoutRef<typeof AccordionPrimitive.Item>;
22-40: Add ARIA label for better accessibility.The accordion trigger should have an ARIA label to improve accessibility.
<AccordionPrimitive.Trigger ref={ref} + aria-label="Toggle accordion" className={cn( "flex flex-1 items-center justify-between py-4 text-sm font-medium transition-all hover:underline text-left [&[data-state=open]>svg]:rotate-180", className, )} {...props} >templates/types/streaming/nextjs/app/components/ui/card.tsx (2)
4-17: Consider using semantic HTML elements for better accessibility.The Card component could use
articleorsectioninstead ofdivfor better semantic meaning.- <div + <article ref={ref} className={cn( "rounded-xl border border-neutral-200 bg-white text-neutral-950 shadow dark:border-neutral-800 dark:bg-neutral-950 dark:text-neutral-50", className, )} {...props} - /> + />
31-41: Consider using heading elements for CardTitle.The CardTitle component should use appropriate heading elements (h1-h6) instead of div.
- <div + <h2 ref={ref} className={cn("font-semibold leading-none tracking-tight", className)} {...props} - /> + />templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx (2)
187-211: Add error boundary and loading states for better UX.The Accordion component should handle loading and error states gracefully.
+import { ErrorBoundary } from 'react-error-boundary'; + +function ErrorFallback({error}: {error: Error}) { + return ( + <div className="text-red-500"> + <p>Something went wrong:</p> + <pre>{error.message}</pre> + </div> + ); +} -<Accordion type="single" collapsible className="space-y-2"> +<ErrorBoundary FallbackComponent={ErrorFallback}> + <Accordion type="single" collapsible className="space-y-2">
194-203: Add keyboard navigation support for better accessibility.The AccordionTrigger should support keyboard navigation.
<AccordionTrigger + role="button" + tabIndex={0} + onKeyDown={(e) => { + if (e.key === 'Enter' || e.key === ' ') { + e.preventDefault(); + e.currentTarget.click(); + } + }} className="hover:bg-accent hover:no-underline py-3 px-3 gap-2" >
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
templates/types/streaming/fastapi/main.py(2 hunks)templates/types/streaming/nextjs/app/components/ui/accordion.tsx(1 hunks)templates/types/streaming/nextjs/app/components/ui/card.tsx(1 hunks)templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx(2 hunks)templates/types/streaming/nextjs/package.json(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- templates/types/streaming/fastapi/main.py
🧰 Additional context used
📓 Path-based instructions (4)
templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
templates/types/streaming/nextjs/app/components/ui/accordion.tsx (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
templates/types/streaming/nextjs/app/components/ui/card.tsx (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
templates/types/streaming/nextjs/package.json (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
🔇 Additional comments (3)
templates/types/streaming/nextjs/app/components/ui/accordion.tsx (1)
1-7: LGTM! Clean imports and client directive.The imports are well-organized, and the "use client" directive is correctly placed at the top.
templates/types/streaming/nextjs/package.json (2)
16-16: Verify compatibility with @llamaindex/chat-ui version.The fixed version
0.0.14of @llamaindex/chat-ui might be outdated. Consider using a more recent version if available.
32-32: Verify peer dependencies for UI libraries.The newly added UI-related packages (
marked,tailwind-merge,tailwindcss-animate) might have peer dependencies that need to be installed.Also applies to: 38-39
✅ Verification successful
All peer dependencies are satisfied ✓
markedandtailwind-mergehave no peer dependenciestailwindcss-animaterequirestailwindcss >= 3.0.0, which is satisfied by the existingtailwindcss@^3.3.6in devDependencies🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Check peer dependencies for new packages npm info marked peerDependencies --json npm info tailwind-merge peerDependencies --json npm info tailwindcss-animate peerDependencies --jsonLength of output: 189
Script:
#!/bin/bash # Check if tailwindcss is present in package.json and its version cat templates/types/streaming/nextjs/package.json | jq '.dependencies.tailwindcss, .devDependencies.tailwindcss'Length of output: 128
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (5)
templates/types/streaming/nextjs/app/components/ui/card.tsx (2)
4-17: Consider enhancing accessibility of the Card component.The Card component should have appropriate ARIA attributes for better accessibility.
const Card = React.forwardRef< HTMLDivElement, React.HTMLAttributes<HTMLDivElement> >(({ className, ...props }, ref) => ( <div ref={ref} className={cn( "rounded-xl border border-neutral-200 bg-white text-neutral-950 shadow dark:border-neutral-800 dark:bg-neutral-950 dark:text-neutral-50", className, )} + role="article" {...props} /> ));
43-53: CardDescription should use paragraph element for better semantics.Using a div for text content is not semantically correct.
const CardDescription = React.forwardRef< - HTMLDivElement, + HTMLParagraphElement, - React.HTMLAttributes<HTMLDivElement> + React.HTMLAttributes<HTMLParagraphElement> >(({ className, ...props }, ref) => ( - <div + <p ref={ref} className={cn("text-sm text-neutral-500 dark:text-neutral-400", className)} {...props} - /> + /> ));templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx (3)
167-183: Consider adding loading states for better UX.The CardHeader shows basic loading states but could benefit from skeleton loading UI for better user experience.
<CardHeader className="space-y-4"> {state.retrieve.state !== null && ( <CardTitle className="flex items-center gap-2"> <Search className="h-5 w-5" /> + <div className="flex-1"> {state.retrieve.state === "inprogress" - ? "Searching..." + ? <div className="flex items-center gap-2"> + Searching + <span className="loading-dots">...</span> + </div> : "Search completed"} + </div> </CardTitle> )} ... </CardHeader>
194-203: Enhance AccordionTrigger accessibility with aria-expanded.The AccordionTrigger should indicate its expanded state for screen readers.
<AccordionTrigger className="hover:bg-accent hover:no-underline py-3 px-3 gap-2" + aria-expanded="false" >
204-208: Consider adding loading state for answer content.The answer content should show a loading state when being fetched.
{question.answer && ( <AccordionContent className="border-t pt-3"> + {question.state === 'inprogress' ? ( + <div className="animate-pulse h-20 bg-neutral-100 dark:bg-neutral-800 rounded" /> + ) : ( <Markdown content={question.answer} /> + )} </AccordionContent> )}
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
templates/types/streaming/nextjs/app/components/ui/accordion.tsx(1 hunks)templates/types/streaming/nextjs/app/components/ui/card.tsx(1 hunks)templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx(2 hunks)templates/types/streaming/nextjs/package.json(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- templates/types/streaming/nextjs/package.json
- templates/types/streaming/nextjs/app/components/ui/accordion.tsx
🧰 Additional context used
📓 Path-based instructions (2)
templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
templates/types/streaming/nextjs/app/components/ui/card.tsx (1)
Pattern templates/**: For files under the templates folder, do not report 'Missing Dependencies Detected' errors.
⏰ Context from checks skipped due to timeout of 90000ms (42)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --example-file)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --no-files)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --no-files)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --example-file)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --no-files)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --no-files)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --example-file)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --no-files)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --no-files)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --llamacloud)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --example-file)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --no-files)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --llamacloud)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --example-file)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --no-files)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --llamacloud)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --example-file)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --no-files)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --llamacloud)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --example-file)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --no-files)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --llamacloud)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --no-files)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --llamacloud)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --example-file)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --no-files)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --no-files)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --example-file)
🔇 Additional comments (1)
templates/types/streaming/nextjs/app/components/ui/card.tsx (1)
75-82: LGTM! Well-organized exports.The exports are well-organized and follow a consistent alphabetical order.
templates/types/streaming/nextjs/app/components/ui/chat/custom/deep-research-card.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.changeset/chilly-bats-smile.md (1)
4-5: Enhance the changeset description with more details.While the description identifies the issue, it would be more helpful to include:
- The specific CORS error that was occurring
- Which file sources were affected
- The solution implemented (CORS middleware addition)
Consider expanding the description like this:
-Fix the error: Unable to view file sources due to CORS. +Fix CORS error preventing file source viewing in development environment. + +- Added CORS middleware to FastAPI application +- Enables cross-origin requests during development +- Resolves issue with accessing file sources from different origins
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.changeset/chilly-bats-smile.md(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (45)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --example-file)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, express, --no-files)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, ubuntu-22.04, nextjs, --no-files)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --example-file)
- GitHub Check: typescript (20, 3.11, windows-latest, express, --no-files)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, windows-latest, nextjs, --no-files)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --llamacloud)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --example-file)
- GitHub Check: typescript (20, 3.11, macos-latest, express, --no-files)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --llamacloud)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --example-file)
- GitHub Check: typescript (20, 3.11, macos-latest, nextjs, --no-files)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --llamacloud)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --example-file)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, express, --no-files)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --llamacloud)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --example-file)
- GitHub Check: typescript (18, 3.11, ubuntu-22.04, nextjs, --no-files)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --llamacloud)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --example-file)
- GitHub Check: typescript (18, 3.11, windows-latest, express, --no-files)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --llamacloud)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --example-file)
- GitHub Check: typescript (18, 3.11, windows-latest, nextjs, --no-files)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --llamacloud)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, express, --no-files)
- GitHub Check: python (20, 3.11, ubuntu-22.04, fastapi, --no-files)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --llamacloud)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --example-file)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --example-file)
- GitHub Check: python (20, 3.11, windows-latest, fastapi, --no-files)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --llamacloud)
- GitHub Check: typescript (18, 3.11, macos-latest, nextjs, --no-files)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --example-file)
- GitHub Check: python (20, 3.11, macos-latest, fastapi, --no-files)
🔇 Additional comments (1)
.changeset/chilly-bats-smile.md (1)
1-3: LGTM! Appropriate version bump for a bug fix.The patch version bump is correctly specified for a CORS-related bug fix.
Summary by CodeRabbit
New Features
Improvements
Bug Fixes