Remove aos-owned surface-check script #507

Merged
coilyco-ops merged 2 commits from issue-505 into main 2026-07-14 01:05:59 +00:00
Member

Removed the aos-owned surface-check script and its ward verb. The remaining role-surface doc now describes the capability map only, and the ward-specs bundle validator no longer expects the deleted tar member.

  • Deleted .ward/surface-check.sh and removed surface-check from .ward/ward.yaml.
  • Updated docs and the bundle validator to stop advertising or packaging the script.
  • Validation: ward doctor, ward --audit-override-dirty exec pre-commit-all.

closes #507
ward.workflow: pull-request-and-merge

Removed the aos-owned surface-check script and its ward verb. The remaining role-surface doc now describes the capability map only, and the ward-specs bundle validator no longer expects the deleted tar member. - Deleted `.ward/surface-check.sh` and removed `surface-check` from `.ward/ward.yaml`. - Updated docs and the bundle validator to stop advertising or packaging the script. - Validation: `ward doctor`, `ward --audit-override-dirty exec pre-commit-all`. closes #507 ward.workflow: pull-request-and-merge
chore: remove aos surface-check script
All checks were successful
ci / ward-doctor (pull_request) Successful in 1m42s
ci / gate (pull_request) Successful in 4m9s
ee233a7678
Merge remote-tracking branch 'origin/main' into issue-505
All checks were successful
ci / ward-doctor (pull_request) Successful in 45s
ci / gate (pull_request) Successful in 1m45s
2cd4cf5d9f
# Conflicts:
#	agentic_os/pre_commit/check_ward_specs_bundle.py
Author
Member

WARDED_WORKFLOW: #507

details

workflow: pull-request-and-merge; review summary: skipped. The in-container review gate was intentionally skipped because the temporary ward default is still pending brokered QA.

The repair felt mechanical. The only merge conflict was the stale validator deletion from #506, and the surface-check removal stayed intact.
Confidence: high.
Surprise: ward doctor passed after the repaired branch hit CI, so the repo-side validation settled cleanly.
Follow-up: merge the existing PR when the director lane is ready.

WARDED_WORKFLOW: https://forgejo.coilysiren.me/coilyco-flight-deck/agentic-os/pulls/507 <details><summary>details</summary> workflow: pull-request-and-merge; review summary: skipped. The in-container review gate was intentionally skipped because the temporary ward default is still pending brokered QA. The repair felt mechanical. The only merge conflict was the stale validator deletion from #506, and the surface-check removal stayed intact. Confidence: high. Surprise: `ward doctor` passed after the repaired branch hit CI, so the repo-side validation settled cleanly. Follow-up: merge the existing PR when the director lane is ready. </details>
Author
Member

WARDED_WORKFLOW review-note

verdict: pass
confidence: high
summary: PR 507 now removes the aos-owned live surface-check script, removes the ward exec surface-check verb, and updates the role-surface and ward-spec docs so AOS only documents the descriptive capability map. The repaired head is mergeable and Forgejo CI is green.
findings: []
conclusion: The director review passes. The PR-head grep for surface-check|surface check|surface_check returned no hits, and the remaining diff is scoped to deleting the script, removing its ward verb, and adjusting docs.

WARDED_WORKFLOW review-note verdict: pass confidence: high summary: PR 507 now removes the aos-owned live surface-check script, removes the `ward exec surface-check` verb, and updates the role-surface and ward-spec docs so AOS only documents the descriptive capability map. The repaired head is mergeable and Forgejo CI is green. findings: [] conclusion: The director review passes. The PR-head grep for `surface-check|surface check|surface_check` returned no hits, and the remaining diff is scoped to deleting the script, removing its ward verb, and adjusting docs.
Sign in to join this conversation.
No description provided.