Merged
Conversation
Logs MCP tool activity and server subprocess output to ~/.buckaroo/logs/ for debuggability. Also accept .parq as a parquet file extension in data_loading. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Extra space before `matrix:` key caused GitHub Actions to reject the workflow file without running any jobs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add OS guard on cache-apt-pkgs-action (Linux only) - Update OS checks from 'ubuntu-latest' to contains(matrix.os, 'ubuntu') to match the depot-ubuntu-latest runner name - Use explicit `bash` shell for script steps on Windows - Add OS to artifact name to avoid upload conflicts Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The docs check was previously never running due to the OS condition mismatch. Now that it runs, it reveals a pre-existing failure. Mark it non-blocking so it doesn't prevent the actual build/test steps. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The Playwright server tests require standalone.js to be present in buckaroo/static/, but full_build.sh only built widget.js. This caused all browser-rendering tests to timeout in CI. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The browser window never opened because of three stacked bugs: 1. MCP tool started server with --no-browser, disabling all browser mgmt 2. focus.py only supported Chrome, not the user's default browser 3. "found" substring matched "not_found", so find always returned True Replace the old webbrowser.open/focus_browser_tab split with a single idempotent find_or_create_session_window() that searches all Chromium browsers (Chrome, Arc, Chromium) via AppleScript, creates a dedicated window if none exists, and falls back to webbrowser.open on non-macOS. Add optional BUCKAROO_APP_MODE=1 for Chrome --app mode (electron-light). Add diagnostic info throughout so issues are immediately visible: - /health returns pid, start time, uptime - /load returns browser_action and server_pid - MCP tool response includes a Server/Browser/Session footer - Server logs to ~/.buckaroo/logs/server.log with timestamps and PIDs - Session ID embedded in page <title> for app-mode window discovery Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- AppleScript: reorder activate→set index so target window ends up on top after Chrome is brought forward - AppleScript: add reload parameter (set URL of t to URL of t) so existing tabs pick up newly loaded data - LoadHandler: push full initial_state (not just metadata) to WS clients so the JS UI updates without a page reload - Add browser focus primitives and rubrics design doc Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Moves the MCP tool module outside the buckaroo package so it doesn't trigger buckaroo/__init__.py imports (which require polars and other heavy optional deps). This fixes the MCP server failing to start when installed via uvx --from "buckaroo[mcp]". Also adds buckaroo_table as a second tool alias for view_data. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The health endpoint now returns uptime_s in addition to status. Use toMatchObject instead of toEqual so the test doesn't break when new fields are added. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Revert unscoped flex utility and gap-removal CSS rules from dcf-npm.css that were applied globally and broke Jupyter Lab rendering (search bar swallowed, status bar overlap). The standalone server fix lives exclusively in handlers.py's inline <style> tag, matching the same approach Jupyter uses with .cell-output-ipywidget-background scoped rules. Also add tsconfig.tsbuildinfo cleanup to full_build.sh to prevent stale TypeScript build cache from producing outdated CSS output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
paddymul
force-pushed
the
feat/mcp-ui-tool
branch
from
15cf2c5 to
957752d
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
1 participant
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.
No description provided.