Skip to content

Use harness service in antigravity implementation.#71

Merged
wjjclaud merged 5 commits into
mainfrom
harnessService-pr2
Jun 9, 2026
Merged

Use harness service in antigravity implementation.#71
wjjclaud merged 5 commits into
mainfrom
harnessService-pr2

Conversation

@wjjclaud

@wjjclaud wjjclaud commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Migrates the built-in Antigravity harness - both the Python server and the Go
client, from the legacy AgentService RPC to the new HarnessService protocol
introduced in PR #69. Update issue #54. Also run formatter to harness_server.py.

Tested

  • Harness server: PYTHONPATH=python:. python python/antigravity/harness_server.py
  • e2e script go run ./cmd/e2e
  • ax client go run -tags=harness ./cmd/ax exec --config internal/ax2.yaml --input "..."

@wjjclaud wjjclaud requested a review from anj-s June 9, 2026 16:58
Comment thread python/antigravity/harness_server.py Outdated
await server.start()
await server.wait_for_termination()


Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There is a lot of linting fixes in this file. Can you either add the linting as a command for the team to run or revert these specific chagnes?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Reverted the lint changes. Will do lint fixes in a separate PR.

Start: &proto.AgentStart{
AgentId: "antigravity",
Messages: inputs,
HarnessId: "antigravity",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

what do we use this for?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is unused for now. Potentially use case is when we have multiple harness implementations in a single ax binary.

@anj-s anj-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add TODOs for anything that is demo/testing related for us to cleanup

Base automatically changed from harnessService to main June 9, 2026 18:28
@wjjclaud wjjclaud merged commit 220ff40 into main Jun 9, 2026
7 checks passed
@wjjclaud wjjclaud deleted the harnessService-pr2 branch June 9, 2026 19:18
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