Consolidate local browser, Agent, video, and plugin updates - #513
Merged
Merged
Conversation
Devin-AXIS
force-pushed
the
codex/local-integration-20261004
branch
3 times, most recently
from
October 4, 2026 12:42
a8ff6db to
0425c5e
Compare
Devin-AXIS
marked this pull request as ready for review
October 4, 2026 12:48
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0425c5efa2
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Devin-AXIS
force-pushed
the
codex/local-integration-20261004
branch
from
October 4, 2026 13:14
0425c5e to
b469bc1
Compare
Devin-AXIS
force-pushed
the
codex/local-integration-20261004
branch
from
October 4, 2026 13:37
b469bc1 to
d904f0e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pending local changes were spread across desktop worktrees, standalone plugin sources, and the personal video component library. This consolidates them into one commit so the development app and packaged release share the current browser, native Agent workflows, video runtime, and plugin catalog.
Validation:
pnpm --filter @ipollowork/app testpassed on both Ubuntu and macOS in final CI; app typecheck and production build passed locally. The final provider recovery regression suites passed all 71 tests, including both failure/recovery cases; the 74 affected session/video tests passed after desktop recovery retirement.pnpm --filter ipollowork-server test— 978 passed, 6 existing environment skips across all 79 files; server typecheck/build passed.pnpm --filter @ipollowork/desktop test— 201 passed, 4 existing environment skips; Electron typecheck and the 57-method bridge check passed. RealcreateMainWindowexecution covers development and packaged entrypoints with saved legacy queue data: one normal window is created, the queue stays untouched, and no polling or hidden delivery worker starts. All 74 affected App/video tests and app typecheck passed.bun run build:local-studio, component/catalog checks, all 19 Shotcraft theme timelines, and all six imported component playback/seek proofs passed.app-smokeand all sixagent-browserproof frames passed. Reproduce withnode evals/runner/run.mjs --flow app-smoke --flow agent-browser --cdp-url http://127.0.0.1:9823against a test profile. Local proof reports are underevals/results/; generated reports are excluded from source control.The original development startup checkout is synchronized to this commit; the final server binding changes load after restarting the development process. Earlier normal-profile inspection verified the browser takeover/decision/identity controls, GPT-6/GPT-6.1 model options, and existing recorder/annotation/Onto plugin entries. The user-profile Cloud endpoint was separately switched to the hosted service at the user's request; hosted sign-in is pending, so a live marketplace installation is not claimed.
This commit is based directly on main dfa728a and preserves its browser and native compaction changes. The previously consolidated source passed all GitHub Ubuntu/macOS checks, i18n and the three DeepSeek Harness Studio builds. Rebase conflict verification passed 307 App tests, 79 server tests and both typechecks; a separate main-behavior audit passed 131 App and 96 server tests. The unchanged Cloud marketplace owners passed all 21 focused tests.
The two review fixes additionally passed 66 HTTP/SQLite work-item and completion tests (560 assertions), server typecheck/build, 99 affected App tests, app and desktop typechecks, and a real isolated Electron/React mounting regression. That regression fails against the previous unkeyed VideoPanel branch and verifies project/workspace/root changes reset state while same-project view/theme changes preserve it. The eight changed files passed the maintainability audit with no errors or warnings. Final amended commit d904f0e passed the complete GitHub Ubuntu/macOS pipeline, including App/server/desktop tests, Electron IPC typecheck and end-to-end checks, plus i18n and all three Studio builds. The tag-only publishing job was skipped as intended. The PR is mergeable and all review threads are resolved.