[pull] main from github:main#116
Merged
Merged
Conversation
… empty CI output (#34740)
…yield and episode metrics (#34750)
…ows (#34753) * Initial plan * Reduce incremental bundle size by excluding base branch objects Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Polish incremental bundle test naming Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Address review feedback for incremental bundle optimization Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Simplify bundle test artifact cleanup Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Harden incremental bundle semantic assertions Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Refine bundle test assertion clarity Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Remove redundant bundle head assertion Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * test: add fallback integration coverage for incremental bundle exclusions Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * test: verify fallback incremental bundle behavior when base ref missing Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Fix incremental base-ref fetch warning and remove redundant probe Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * chore: plan dry solid git audit Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Refactor remote-ref checks for DRY while preserving git bundle behavior Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Harden helper error handling in bundle ref probe refactor Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…add missing log parser (#34768) * Initial plan * Fix smoke-antigravity: replace npm install with GCS binary download, add log parser Root cause 1: @google/antigravity-cli does not exist on npm - the Antigravity CLI is a Go binary distributed from Google Cloud Storage. Replace with a shell script that downloads from storage.googleapis.com/antigravity-public/antigravity-cli/. Root cause 2: parse_antigravity_log.cjs was missing. Create the log parser for Antigravity's stream-json JSONL format. Also fix DefaultAntigravityVersion from 0.39.1 (incorrect, copied from Gemini) to 1.0.2-6113393518706688 (actual current version). Recompile smoke-antigravity.lock.yml with the corrected steps. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Fix review comments: mark antigravity experimental, remove Node.js setup, fix shell injection, add tests Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
…ent logs (#34782) * wip: start unified timeline implementation Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * wip: plan unified timeline with agent events + JS + Go rendering surfaces Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * feat: unified event timeline for MCP Gateway, AWF firewall, and agent logs - Add TimelineSourceAgent + agent event kinds (agent_turn, agent_tool_start, agent_tool_done) to gateway_logs_timeline.go - Add parseEventsJSONL helper and collectAgentTimelineEvents in same file - Update BuildUnifiedTimeline to include agent events from events.jsonl - Add rendering primitives for agent events in gateway_logs_timeline_render.go (renderAgentTurnRow, renderAgentToolStartRow, renderAgentToolDoneRow) - Update renderUnifiedTimeline summary to show AG counts - Wire renderAuditUnifiedTimeline into audit.go (Go audit log generation surface) - Wire displayUnifiedTimeline into logs_orchestrator.go (Go logs CLI surface) - Create actions/setup/js/unified_timeline.cjs: JS parser + renderer for GITHUB_STEP_SUMMARY (all three sources, sorted merge, Markdown <details> table) - Create actions/setup/js/unified_timeline.test.cjs: 66 Vitest tests - Integrate generateUnifiedTimelineSummary into writeStepSummaryWithTokenUsage in parse_mcp_gateway_log.cjs so timeline is appended to every step summary - Create pkg/cli/gateway_logs_timeline_test.go: Go unit tests for all surfaces Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * fix: address code review issues from parallel_validation - Guard timelineSourceLabel default case against short strings (Go + JS) - Deduplicate sorted-order check in displayUnifiedTimeline (calls sortUnifiedTimelineEvents once instead of inline-checking then calling) - Fix sortUnifiedTimelineEvents comment to reflect sort.SliceStable usage; inline the sort and remove now-unused sortEventsStable helper - Use truncate() helper for firewall detail in unified_timeline.cjs for consistent ellipsis handling with other collectors - Fix test assertion: replace buggy regex with plain string check for pipe escaping in buildUnifiedTimelineMarkdown Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * docs(adr): add draft ADR-34782 for unified event timeline * Update; rm -rf / Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * fix: address PR review comments - JS firewall filters, Go header, lock file revert Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * fix: simplify JS url field lookup to use lowercase json tag Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
* Initial plan * Add !integration go:build tags to missing unit test files Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
* Initial plan * doc: document tracker-id frontmatter field in main reference Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * doc: trim tracker-id section to remove bloat Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…34804) * Set Codex default model constant to gpt-5.4 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Recompile workflows for codex default model gpt-5.4 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )
Summary by cubic
Bumps default firewall/gateway versions, fixes
gh aw upgradeon GHE, and reduces incremental PR bundle size by excluding base-branch history. Recompiled workflow lockfiles and trimmed docs to match the new defaults.Dependencies
gh-aw-firewallv0.25.55; updategh-aw-mcpgv0.3.19; sync embedded AWF schema.Bug Fixes
gh aw upgradewith GHE auth: always fetch releases fromgithub.comand rungh extensioncommands withGH_HOST=github.com.origin/<base_branch>objects to shrink bundles while preserving the same tip.Written for commit 5fa30ed. Summary will update on new commits. Review in cubic