Skip to content

feat(web) (4/5): skill authoring workbench - #5

Open
zjncs wants to merge 1 commit into
feat/authoring-fix-publishfrom
feat/authoring-web
Open

zjncs wants to merge 1 commit into
feat/authoring-fix-publishfrom
feat/authoring-web

Conversation

@zjncs

@zjncs zjncs commented Oct 14, 2026

Copy link
Copy Markdown
Owner

Layer (d) of the authoring platform split (iflytek#934 on iflytek/skillhub), stacking on layers (a)–(c). (Recreated after the previous stacked PR was auto-closed by a branch mishap — same content.)

The web workbench under /dashboard/authoring: draft list + detail with unsaved-state tracking and optimistic-concurrency conflicts, runtime binding form, run detail with live event console (SSE + polling fallback), finding cards with suggested-fix diff preview and one-click apply/dismiss, submit dialog, en/zh/ru copy, Playwright e2e. Verified: typecheck clean, 922 web unit tests pass (only the two pre-existing upstream theme tests fail, identically on plain main).

Copilot AI balanced review requested due to automatic review settings October 14, 2026 14:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@zjncs
zjncs force-pushed the feat/authoring-web branch from 058822e to 0b8fbe0 Compare October 14, 2026 14:45
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch 2 times, most recently from 65c048d to fa41aef Compare October 14, 2026 14:55
@zjncs
zjncs force-pushed the feat/authoring-web branch 2 times, most recently from 5f1d6f9 to be1e677 Compare October 14, 2026 15:08
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch 2 times, most recently from b03d08d to cb73e2f Compare October 15, 2026 06:13
@zjncs
zjncs force-pushed the feat/authoring-web branch 3 times, most recently from 96ae4b9 to fcfcbb8 Compare October 15, 2026 16:58
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from cb73e2f to 397b1ff Compare October 15, 2026 16:58
@zjncs
zjncs force-pushed the feat/authoring-web branch from fcfcbb8 to 16668ac Compare October 17, 2026 03:38
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from 397b1ff to f646210 Compare October 17, 2026 03:38
@zjncs
zjncs force-pushed the feat/authoring-web branch from 16668ac to 4b63b5d Compare October 21, 2026 08:58
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from f646210 to 51d3deb Compare October 21, 2026 08:58
@zjncs
zjncs force-pushed the feat/authoring-web branch from 4b63b5d to 55ad81b Compare October 22, 2026 05:42
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from 51d3deb to b55a143 Compare October 22, 2026 05:42
Layer (d) of the authoring platform (iflytek#934), stacking on the fix/publish
layer: the web workbench under /dashboard/authoring —

- draft list and draft detail (files / runtime binding / validation runs)
  with an editor that tracks unsaved state and surfaces optimistic-
  concurrency conflicts
- runtime binding form switching fields by agent type (interpreter vs
  endpoint/model), tool allowlist and MCP server declarations with
  immediate validation
- run detail: status card, live event console (SSE with a 2s polling
  fallback and cursor replay), finding cards with severity, location,
  suggested-fix diff preview and one-click apply/dismiss
- submit dialog handing validated drafts to the publish pipeline
- copy in en/zh/ru; Playwright e2e for the full authoring flow
  (e2e/authoring-flow.spec.ts)

Verified: typecheck clean; 922 web unit tests pass (the only failures
in the suite are the two pre-existing upstream theme tests, which fail
identically on plain main).

Signed-off-by: zjncs <18910855655@163.com>
@zjncs
zjncs force-pushed the feat/authoring-web branch from 55ad81b to 3d3f42c Compare October 23, 2026 10:40
@zjncs
zjncs force-pushed the feat/authoring-fix-publish branch from b55a143 to 43a0788 Compare October 23, 2026 10:40
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.

2 participants