Skip to content

feat(demo): flowgram agent - #1043

Draft
louisyoungx wants to merge 71 commits into
bytedance:mainfrom
louisyoungx:feat/flowgram-agent
Draft

louisyoungx wants to merge 71 commits into
bytedance:mainfrom
louisyoungx:feat/flowgram-agent

Conversation

@louisyoungx

@louisyoungx louisyoungx commented Dec 25, 2025 •

Copy link
Copy Markdown
Collaborator

20260119-120026

louisyoungx and others added 15 commits December 26, 2025 16:17
- Introduce Loop, Break, and Continue nodes with full CRUD tooling
- Allow Code, Condition, HTTP, and LLM nodes to be created inside Loop containers via parentNodeID
- Enforce same-container edge creation and validate node connectivity rules
- Replace legacy tool-call placeholders with <system_note> tags and update system prompt
- Remove completed-count from todo render to reduce noise
@louisyoungx louisyoungx changed the title feat(demo): flowgram agent WIP: feat(demo): flowgram agent Jan 6, 2026
@louisyoungx
louisyoungx marked this pull request as draft January 6, 2026 11:40
@louisyoungx louisyoungx changed the title WIP: feat(demo): flowgram agent feat(demo): flowgram agent Jan 20, 2026
@huxinfeng

huxinfeng commented Mar 12, 2026 •

Copy link
Copy Markdown

为什么这个pull还没有合入主分支

This text is auto-translated:
Why hasn’t this pull been merged into the main branch yet?

@xiamidaxia

xiamidaxia commented Mar 13, 2026 •

Copy link
Copy Markdown
Collaborator

为什么这个pull还没有合入主分支

This text is auto-translated:
Why hasn’t this pull been merged into the main branch yet?

这个提交改动比较大,demo 也会变重,需要从主 demo 里独立出来不过暂时还没有时间去跟进,如果你们有兴趣也可以直接参考该分支

This text is auto-translated:

Why has this pull not been merged into the main branch yet?

This text is auto-translated:
Why hasn’t this pull been merged into the main branch yet?
The changes in this submission are relatively large, and the demo will also become heavier. It needs to be independent from the main demo. However, I don’t have time to follow up yet. If you are interested, you can refer to this branch directly.

@squallliu

squallliu commented Apr 15, 2026 •

Copy link
Copy Markdown
Contributor

你们自家的semi design不是有ai组件了吗,咋还引入了antd

This text is auto-translated:
Doesn’t your own semi design have AI components? Why did you also introduce antd?

@louisyoungx louisyoungx left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

[P1] Remove the hardcoded provider API key from the browser demo

The new createAgentPlugin configuration in apps/demo-free-layout/src/hooks/use-editor-props.tsx embeds a non-placeholder API key alongside the provider baseURL. This configuration is shipped in the browser bundle, so anyone loading the demo can extract the credential and make requests billed to that account. Please remove the credential, revoke/rotate it if it is real, and require users to supply their own key or route authenticated requests through a server. I have intentionally not reproduced the key in this review. This PR is also still a draft and conflicts with main.

config: {
// Default config - users should update via UI settings
// 默认配置 - 用户应通过 UI 设置更新
apiKey: 'e96f48f1-daf9-447e-8258-5003bfbffd0f',

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

[P1] This default configuration embeds a provider API key in a publicly served browser bundle. Remove the credential and use user-supplied configuration or a server-side authenticated proxy; rotate/revoke the key if it is real. The key is deliberately omitted from this comment.

This branch has not been deployed

No deployments
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.

4 participants