feat: support editable class labels in question classifier#35430
Conversation
|
Hi! I'm I would like to apply some automated changes to this pull request, but it looks like I don't have the necessary permissions to do so. To get this pull request into a mergeable state, please do one of the following two things:
|
63f111f to
b1a6d66
Compare
|
Heads-up on merge order for this feature: this PR depends on the runtime/schema changes landing in Graphon first. Even though the Dify-side UI is ready, we should wait until the companion Graphon PR is merged and a new Graphon version is released before merging this PR. After that, this PR should be updated to bump Without that release order, users can see/edit the new classifier labels in the UI, but workflow execution may still run against an older Graphon runtime that does not fully support the new So the intended sequence is:
Companion Graphon PR: https://github.com/langgenius/graphon/pull/new/blackoutta/question-classifier-labels-graphon |
7614bd5 to
92a2858
Compare
Important
Fixes #<issue number>.Summary
Fixes #35429
Refs #33806
Supersedes #34075
Companion Graphon branch:
langgenius/graphon:blackoutta/question-classifier-labels-graphonCompanion Graphon PR URL: https://github.com/langgenius/graphon/pull/new/blackoutta/question-classifier-labels-graphon
This recreates the Dify-side portion of editable Question Classifier labels after the workflow runtime moved into
graphon.The UI now treats Question Classifier branch data as three separate concepts:
id: stable branch identifier for routing/handlesname: classifier-facing category description used for prompt logic and preserved asclass_namelabel: user-facing branch title shown in the editor, canvas, and previewIncluded changes:
class_namesemantics for downstream descriptive outputclass_labelin Question Classifier output configurationlabelwith sensibleCLASS NfallbacksThis branch was validated locally against an editable Graphon checkout wired through
uv.From Codex
Screenshots
CLASS Nin the UI.Checklist
make lint && make type-check(backend) andcd web && pnpm exec vp staged(frontend) to appease the lint gods