Skip to content

feat(samples): langchain#451

Open
blove wants to merge 1 commit intomainfrom
blove/feat/samples-langchain
Open

feat(samples): langchain#451
blove wants to merge 1 commit intomainfrom
blove/feat/samples-langchain

Conversation

@blove
Copy link
Contributor

@blove blove commented Jan 22, 2026

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[x] Other... Please describe:

What is the new behavior?

Creates new langchain sample apps

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@blove blove force-pushed the blove/feat/samples-langchain branch from f1843c3 to 9368c37 Compare January 22, 2026 23:47
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f1843c3589

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

import { Message } from './Message';
import { ScrollArea } from './scrollarea';
import { Textarea } from './textarea';
import { Textarea } from './Textarea';

Choose a reason for hiding this comment

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

P2 Badge Import Textarea with correct case

This import uses ./Textarea, but the file on disk is textarea.tsx (lowercase). On case-sensitive filesystems (Linux CI, most prod builds), the module resolution will fail and the sample won't compile. Consider reverting to ./textarea or renaming the file to match.

Useful? React with 👍 / 👎.

import { RichMessage } from './RichMessage';
import { ScrollArea } from './scrollarea';
import { Textarea } from './textarea';
import { Textarea } from './Textarea';

Choose a reason for hiding this comment

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

P2 Badge Import Textarea with correct case

Same case-sensitivity issue here: ./Textarea does not match the actual textarea.tsx filename, so builds on Linux will error with “Cannot find module”. Align the import with the filename or rename the file.

Useful? React with 👍 / 👎.

@blove blove force-pushed the blove/feat/samples-langchain branch from 9368c37 to 41c5f0f Compare February 24, 2026 18:26
@blove blove force-pushed the blove/feat/samples-langchain branch from 41c5f0f to c916991 Compare February 26, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant