Skip to content

ci: enforce repository formatting#62

Merged
WH-2099 merged 1 commit into
mainfrom
ci/61-enforce-repository-formatting
Jul 18, 2026
Merged

ci: enforce repository formatting#62
WH-2099 merged 1 commit into
mainfrom
ci/61-enforce-repository-formatting

Conversation

@WH-2099

@WH-2099 WH-2099 commented Jul 18, 2026

Copy link
Copy Markdown
Member

Closes #61

Summary

  • Enforce vp fmt --check through both just check and pull request CI.
  • Apply the current vp fmt --write output across source and test files.

Why

Pull request checks previously allowed formatting drift.

Impact

The only behavioral change is the new formatting gate.

The remaining file changes are generated formatting updates.

Validation

  • vp fmt . --check
  • just check
  • 1060 passing tests, 20 credential-gated skips, and 0 failures.
  • Driver artifact existence, size, executable bit, and Bun shebang checks.

@WH-2099 WH-2099 self-assigned this Jul 18, 2026
@WH-2099
WH-2099 marked this pull request as ready for review July 18, 2026 21:12
Copilot Bot review requested due to automatic review settings July 18, 2026 21:12
@WH-2099
WH-2099 enabled auto-merge (squash) July 18, 2026 21:12
@WH-2099
WH-2099 merged commit 8de1bf3 into main Jul 18, 2026
@WH-2099
WH-2099 deleted the ci/61-enforce-repository-formatting branch July 18, 2026 21:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Enforces repository-wide Vite+ formatting checks locally and in pull request CI while applying generated formatting updates.

Changes:

  • Adds vp fmt --check to just check and PR CI.
  • Reformats source and test files without behavioral changes.

Reviewed changes

Copilot reviewed 50 out of 51 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/pr.yml Adds the CI formatting gate.
justfile Adds formatting verification to check.
src/core/agent-driver-kernel.ts Applies generated formatting.
src/core/driver-command-dispatcher.ts Applies generated formatting.
src/core/driver-permission-broker.ts Applies generated formatting.
src/core/driver-process.ts Applies generated formatting.
src/core/driver-runtime-io.ts Applies generated formatting.
src/core/driver-runtime-timing.ts Applies generated formatting.
src/host-ports/index.ts Applies generated formatting.
src/infrastructure/runtime/driver-instance-socket.ts Applies generated formatting.
src/runtime-events/index.ts Applies generated formatting.
src/runtimes/acp/acp-configuration.ts Applies generated formatting.
src/runtimes/acp/acp-driver-backend.ts Applies generated formatting.
src/runtimes/acp/acp-event-translator.ts Applies generated formatting.
src/runtimes/acp/acp-session-events.ts Applies generated formatting.
src/runtimes/acp/acp-session-setup.ts Applies generated formatting.
src/runtimes/acp/acp-terminal-manager.ts Applies generated formatting.
src/runtimes/acp/v1-contract-adapter.ts Applies generated formatting.
src/runtimes/claude/agent-sdk-event-writer.ts Applies generated formatting.
src/runtimes/claude/agent-sdk-message-events.ts Applies generated formatting.
src/runtimes/claude/agent-sdk-message-translator.ts Applies generated formatting.
src/runtimes/claude/agent-sdk-query-options.ts Applies generated formatting.
src/runtimes/claude/contract-adapter.ts Applies generated formatting.
src/runtimes/contract-projection.ts Applies generated formatting.
src/runtimes/driver-event-publisher.ts Applies generated formatting.
src/runtimes/openai/app-server-driver-backend.ts Applies generated formatting.
src/runtimes/openai/app-server-item-events.ts Applies generated formatting.
src/runtimes/openai/contract-adapter.ts Applies generated formatting.
src/surfaces/cma-sdk/index.ts Applies generated formatting.
src/utils/async.ts Applies generated formatting.
tests/acp-client-request-handler.test.ts Reformats tests.
tests/acp-configuration.test.ts Reformats tests.
tests/acp-event-translator.test.ts Reformats tests.
tests/acp-provider-fixtures.test.ts Reformats tests.
tests/acp-v1-contract-adapter.test.ts Reformats tests.
tests/agent-driver-kernel.test.ts Reformats tests.
tests/async.test.ts Reformats tests.
tests/claude-agent-sdk-driver-backend.test.ts Reformats tests.
tests/claude-agent-sdk-json.test.ts Reformats tests.
tests/claude-agent-sdk-live.test.ts Reformats tests.
tests/claude-agent-sdk-provider-fixtures.test.ts Reformats tests.
tests/claude-contract-adapter.test.ts Reformats tests.
tests/contract-projection.test.ts Reformats tests.
tests/driver-event-publisher.test.ts Reformats tests.
tests/driver-instance-socket.test.ts Reformats tests.
tests/driver-permission-broker.test.ts Reformats tests.
tests/driver-runtime-boundary.test.ts Reformats tests.
tests/openai-app-server-event-bridge.test.ts Reformats tests.
tests/openai-app-server-live.test.ts Reformats tests.
tests/openai-app-server-startup.test.ts Reformats tests.
tests/openai-contract-adapter.test.ts Reformats tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

ci: enforce repository formatting

2 participants