Skip to content

docs: standalone heal service spec - #21

Merged
philmcneely merged 1 commit into
mainfrom
docs/heal-service-spec
Nov 7, 2026
Merged

philmcneely merged 1 commit into
mainfrom
docs/heal-service-spec

Conversation

@philmcneely

Copy link
Copy Markdown
Owner

Adds docs/HEAL_SERVICE_SPEC.md, the spec for one heal service that this framework and the TypeScript framework both call over HTTP, and links it from the README.

It records the decisions made on 2026-11-05:

  • Repo: the service repo stays private until it is tested, commented and documented.
  • Code: it reuses the existing locator-healing code instead of a rewrite.
  • Fingerprint file: .heal/fingerprints.json is kept out of git by default and cached in CI.
  • Test ids: the client sends the project's test-id attribute.
  • Structural or positional locators: a heal whose only unique locator is structural or positional ends needs_human in phase 1.

Docs only; no code changes.

The spec for one heal service that both this framework and the TypeScript
framework will call over HTTP. Records the 2026-11-05 decisions: private repo
until tested and documented, reuse of the existing locator-healing code,
fingerprint file kept out of git by default.
Copilot AI balanced review requested due to automatic review settings November 6, 2026 02:37

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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@philmcneely

Copy link
Copy Markdown
Owner Author

No changes made. Copilot's only review on this PR says it could not run because the reviewer's quota ran out. That review has no code or inline comments, and the PR has no other review comments, so there is nothing to fix. To get a real review, re-request Copilot at a lower effort level, or after the quota resets.

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

✅ All smoke tests passed! View report

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

✅ All chromium login tests passed! View report

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

✅ All firefox login tests passed! View report

@philmcneely
philmcneely requested a balanced review from Copilot November 7, 2026 04:57

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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@philmcneely

Copy link
Copy Markdown
Owner Author

No changes made again. Copilot's second review attempt also failed on quota, the same as the first, and contains no feedback. This PR (README.md, docs/HEAL_SERVICE_SPEC.md) still has no inline or review comments to address, and smoke/chromium/firefox CI is green. Re-request the review once quota resets or at a lower effort level.

@philmcneely
philmcneely merged commit 53cac8f into main Nov 7, 2026
11 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.

2 participants