Skip to content

Integrate Agent Contract 2.1 and Appium lifecycle - #47

Merged
houlianpi merged 2 commits into
mainfrom
feature/agent-integration-v2
Oct 30, 2026
Merged

houlianpi merged 2 commits into
mainfrom
feature/agent-integration-v2

Conversation

@houlianpi

Copy link
Copy Markdown

Requirement

Integrates accepted capabilities #30 and #45 into main after Human Product Owner native-Windows acceptance.

User-visible result

main receives Agent Contract 2.1.0 with the immutable official Appium Skills snapshot and the corrected Appium Runtime session lifecycle. Setup remains the single public entrypoint; Runtime teardown destroys only its owned sessions and never starts replacement Appium servers.

Scope

  • Merge the two reviewed integration commits already accepted on feature/agent-integration-v2:
  • Preserve Contract identity 2.1.0 / 7ef4c25a1659505ff6dce335196c0516ca36709ca30424669b927fc5675d0094 and Runtime/CLI pins.
  • Carry forward pinned Appium Skills provenance, approval boundaries, clean-host rejection, session ownership, bounded shutdown, and redacted diagnostics.

Non-goals

Interface changes

Acceptance scenarios

Given the reviewed integration branch and accepted native Windows evidence
When it is merged into main
Then main contains exactly the accepted #30 and #45 capability commits
And all required PR checks remain green
And no package or Release is published

Verification evidence

Check Result Evidence
Capability #30 tests PASS Installed wheels: Contract 207, CLI 34, Appium 56, Windows 47 plus native-platform skips; final capability CI 26/26.
Capability #30 prompt eval PASS Candidate 13/13; zero regressions; exact installed comparator passed.
Capability #45 tests PASS Clean installed-wheel suite 71 passed; lifecycle CI 6/6.
Independent reviews PASS Both capabilities received independent code and security APPROVE verdicts on their final commits.
Platform acceptance PASS macOS, physical Android/iOS evidence linked on #26/#30; Human Product Owner reports native Windows acceptance completed and passed.

Agent roles

Security review

  • Secrets and sensitive inputs are not written to logs, generated code, or tracked configuration.
  • File writes are scoped and atomic.
  • Command/process inputs are not shell-interpolated.
  • Package downloads and remote instructions are pinned or verified.
  • Not security-sensitive; explanation:

Both included capability diffs passed independent security review. This PR adds no new code beyond those accepted commits.

Compatibility impact

Native macOS and native Windows remain supported. Appium official-helper CI covers Ubuntu, macOS, and Windows with Node 20/22/24. Physical Android/iOS and real macOS evidence are linked on the capability issues. Node 26 remains Action Required.

Rollback

Revert the integration merge commit before any publication. No migration or release side effect is performed by this PR.

Known limitations

Definition of Done

  • The included issues met Definition of Ready before implementation.
  • Scope and non-goals were preserved.
  • Capability acceptance scenarios passed in clean environments.
  • Main-target PR checks pass.
  • Independent reviewers approved both included capabilities.
  • Independent acceptance evidence is linked.
  • Security review passed for both included capabilities.
  • Prompt changes include an eval comparison.
  • Documentation and machine-readable configuration agree.

* feat: delegate Appium setup to official skills

* fix: exercise native Windows CLI path
@houlianpi

Copy link
Copy Markdown
Author

Main-integration gate note:

  • All newly scheduled Ubuntu, Windows, registry, extension, documentation, and syntax checks passed.
  • The new macOS jobs were never assigned a runner (runner_name empty, steps empty). GitHub cancelled the first attempts after 15–20 minutes in queue; reruns remain queued with no runner. This is infrastructure unavailability, not an executed test failure.
  • The exact included capability commits already passed their macOS matrices on PR Delegate Appium setup to official Skills #33 and PR Fix Runtime-owned Appium session teardown #46, with independent review and acceptance recorded there.
  • Human Product Owner reports native Windows acceptance passed and explicitly authorized merge to main for continued testing.

Accordingly, the merge uses the repository administrator path to bypass only the unstarted macOS queue entries. No executed failing check is being waived. No package publication or Release is performed.

@houlianpi
houlianpi merged commit f7d11bd into main Oct 30, 2026
41 of 51 checks passed
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.

1 participant