2 Commits

Include renames

Author	SHA1	Message	Date
3 people	75693e95f4	[OPEND-3114] feat(od-next): open registered agents and CLI BYOK (#8702) ... * feat(od-next): open registered runtimes and CLI BYOK Use the effective runtime registry for OD Next admission and remove retired capability qualification. Keep native observation and existing resume/cold recovery, with the evidence graph owned by observability contracts. Add run-scoped BYOK connection flags with caller-selected environment key input and reuse the existing provider bridge. Accept equivalent frozen Skill JSON and Base64 representations while preserving content and ownership boundaries. Validated with focused contracts, runtime, public Run and CLI regressions; eight real native agents and a real BYOK request/production pair; guard and workspace typecheck. Refs #8701. * fix(web): describe BYOK project file tools Explain that API-provider project runs use OpenCode file tools and require OpenCode to be available. Update the existing notice in all 19 locales and its component/browser regressions. Validated with red/green notice checks, real settings and preview follow-ups, guard and workspace typecheck. Refs #8701. * fix(od-next): preserve hidden prototype form feedback Keep HTML hidden semantics within the shared layout layer without overriding authored product CSS or hidden-until-found behavior. This fixes name errors remaining visible after a successful registration. Validated with a red/green Chromium regression, three resource-boundary checks, a real Claude follow-up with desktop/mobile form interaction, guard and workspace typecheck. Refs #8701. * test(od-next): make Run fixtures select their execution path Explicitly select ordinary Runs for retry persistence, plain stdout replay and the compatibility single-run telemetry case. Keep task hierarchy and terminal fallback telemetry cases active, and assert the selected path. Restore owner write permission to the leftover-agent fixture's frozen directories only during teardown so recursive deletion works on POSIX. Skip symlinks and preserve cleanup errors and every original behavior check. Reproduced all four CI failures before the fix. Focused daemon and E2E checks passed (12 tests), along with guard, workspace typecheck and focused LSP diagnostics. Refs #8701. * fix(daemon): avoid passing Pi resource directories as prompt files Pi reads existing --append-system-prompt paths as files. Stop passing skills, design systems and task input directories there; their resource paths already travel in the composed RPC prompt. Validated with 104 focused runtime/RPC tests, a real Pi 1.1.0 offline startup red/green probe, workspace typecheck and guard. * fix(web): coalesce streamed calls in generation progress Fold in-flight snapshots and terminal inputs through the existing tool-event reducer before rendering the Design Files progress feed. Keep the latest input, stable call identity and the limit of twelve distinct calls. Reproduced the duplicate React key warning before the fix. Runtime and progress-feed regressions now pass alongside existing in-flight tool-row tests. * fix(web): preserve forms and browser storage in prototype previews Permit client-side form submission in opaque interactive previews. Route Web Storage artifacts through the existing isolated powered preview, including daemon hints for large files, and retain guards when isolation is unavailable. Verified URL and srcDoc form interaction, persisted values after reload, and host isolation with focused Chromium regressions. Focused viewer/runtime and daemon tests, workspace typecheck and guard passed. * test(web): keep srcDoc fixtures on the opaque preview path Use an existing non-storage sandbox-shim signal in transport lifecycle fixtures now that Web Storage artifacts select the isolated preview origin. Preserve every reload, activation, recovery and manual-save assertion. Validated all 338 cases in the three affected FileViewer test files, workspace guard and typecheck, and focused diagnostics. * fix(byok): accept keyless IPv6 loopback providers Recognize the bracketed IPv6 hostname returned by WHATWG URL parsing in both settings validation and the OpenCode BYOK runtime. Keep remote hosts key-required. Validated with focused daemon and Web tests, settings browser checks, workspace typecheck and guard. * fix(daemon): restore Pi sessions and preserve completed RPC turns Load prior Pi history with switch_session and capture the authoritative sessionFile through nonblocking get_state. Treat a host-requested SIGTERM after a clean completed turn as shutdown, including Pi builds that report exit 143, while preserving real errors and cancellation. Validated with focused protocol and public Run lifecycle tests, a real Pi generation and native resumed edit, workspace typecheck and guard. * fix(preview): isolate project storage and preserve hidden controls Give powered previews stable per-project loopback origins and reserve those hosts for matching powered GETs before API or static exceptions. Advertise the capability so newer Web clients retain the opaque fallback with older daemons. Reuse the active powered transport in Present. Share a low-specificity hidden-control guard across URL and srcDoc previews without overriding explicit author rules or until-found; preserve doctype mode when synthesizing a missing head. Validated with actual two-project storage, Workers, reload, Present and API isolation browser witnesses, form/hidden regressions, focused Web and daemon checks, workspace typecheck and guard. * test(e2e): align public canaries with best-effort OD Next Read the remaining recipe identity marker without the retired serialized package hash. Make the fake OpenCode help probe finite. Exercise the current public question-form action, completed question turns/new answer tasks, and no-artifact normal endings while retaining cancellation, artifact and deck assertions. Validated with six fake-runtime cases and focused active/off public browser canaries with explicit rollout flags; no production completion policy or timeout was relaxed. * fix(od-next): guide focus restoration after list updates Prefer native modal lifecycle and restore focus to a current control after list rerenders, falling back to the primary action when a filtered item disappears. Real OpenCode and Pi prototypes exposed detached-trigger focus loss. Keep this as best-effort build guidance, without output contracts or automatic repair rounds. Validated the actual artifacts through normal public edit tasks and browser interaction checks; profile resource tests passed. * fix(od-next): retain focus through inline task updates Extend the existing live-control guidance to inline list actions. Real Copilot and Qoder boards also lost keyboard focus when status updates rebuilt their cards; hidden filtered items need a visible fallback. * fix: align Discovery integration tests and presentation storage * test(daemon): tolerate transient memory origin cleanup races --------- Co-authored-by: zhouzheng <zhouzheng@refly.ai>	2026-10-22 05:14:59 +00:00
Golden Obsidian LLCandGitHub	aa0616062d	refactor(daemon): introduce HTTP Request Adapter + typed Deps (#2636) ... * refactor(daemon): introduce HTTP Request Adapter + typed Deps (proof on active-context-routes) Adds a typed HTTP boundary Adapter under apps/daemon/src/http/ that replaces the untyped ServerContext service-locator pattern (30+ fields, mostly any) for route handlers. Routes become pure (input, deps) -> Result<output> functions, unit-testable without Express or supertest. Six new modules under apps/daemon/src/http/: - types.ts Result<T,E>, ok(), err(), JsonRouteSpec, Handler, RouteInputContext, HttpMethod, InputParser - parse.ts rawInput(req), validationError(message, issues?) - response.ts sendJson(), sendApiError(), statusForError() + ERROR_STATUS_BY_CODE map - origin-guard.ts guardSameOrigin(req, origin) wrapping isLocalSameOrigin as a Result - adapter.ts defineJsonRoute(), mountJsonRoute() (only place that knows about req/res) - index.ts barrel active-context-routes.ts migrated as proof of pattern. parsePostActive(), handlePostActive(), handleGetActive() are now pure functions; postActiveRoute and getActiveRoute are exported route specs. The wire signature registerActiveContextRoutes(app, ctx) is preserved so server.ts is untouched. Spec at specs/current/daemon-http-adapter.md captures the strangler migration order for the remaining route files (mcp-routes, chat-routes, artifact routes, etc.) and a StreamRoute follow-up where the Run Orchestrator lands. Wire-format note: cross-origin response moves from the legacy { error: 'cross-origin request rejected' } shape to the structured { error: { code: 'FORBIDDEN', message: ... } } shape. Backwards-compatible via the existing CompatibleErrorResponse = ApiErrorResponse | LegacyErrorResponse union in @open-design/contracts. Validation: - pnpm install (post-rebase, exit 0) - pnpm --filter @open-design/daemon typecheck (both tsconfig.json and tsconfig.tests.json silent => pass) - pnpm --filter @open-design/daemon test: 15 new tests pass (tests/http/adapter.test.ts + tests/active-context-routes.test.ts). 84 pre-existing failures across 23 files are unchanged and unrelated to this PR (Windows symlink / short-name / colon-in-filename, upstream behavior drift, missing plugin marketplace fixtures, and a freshly- added tools-connectors-cli suite of 38 failures that landed during the rebase). Sharpens W4/W5 of specs/current/maintainability-roadmap.md and unlocks W6 (Run Orchestrator). * chore: add core-js, electron-winstaller, protobufjs, sharp to pnpm.onlyBuiltDependencies	2026-05-22 15:20:15 +08:00