coilyco-flight-deck/agentic-os
Watch
2
0
Code Issues 67 Pull requests Projects Releases 588 Packages 1 Activity Actions

Stop requiring manual WARD_CONFIG_REF overrides for warded surfaces #426

Closed
coilyco-ops wants to merge 1 commit from issue-422 into main
pull from: issue-422
merge into: coilyco-flight-deck:main
Conversation 2 Commits 1 Files changed 7 +99 -11
coilyco-ops commented 2026-07-10 09:45:52 +00:00
Owner
Copy link

closes #422

ward.workflow: pull-requests-and-merge

closes #422 ward.workflow: pull-requests-and-merge
coilyco-ops commented 2026-07-10 09:46:58 +00:00
Author
Owner
Copy link

verdict: block
confidence: high
summary: The patch improves host shell entrypoints, but it does not fix the read-only director surface where the problem was observed. In this current director surface both WARD_CONFIG_REF and AOS_REPO_ROOT are unset unless the command is manually prefixed.
findings:

  • severity: high
    file: shell/common.sh
    line: 16
    problem: The implementation only derives WARD_CONFIG_REF when shell/common.sh is sourced, and the shell entrypoint changes only cover shell/bashrc and shell/zshrc.
    impact: The failing surface is a warded read-only director container. That session is not getting either WARD_CONFIG_REF or AOS_REPO_ROOT from these rc files, so warded list --json and warded engineer ... still require the manual inline override in the actual surface this issue is about.
  • severity: medium
    file: shell/common.sh
    line: 21
    problem: _siren_aos_repo_root falls back to git rev-parse --show-toplevel before the canonical aos checkout.
    impact: If common.sh is sourced without AOS_REPO_ROOT while the current directory is another repo, it can build an agentic-os WARD_CONFIG_REF using a non-agentic-os commit SHA.
    conclusion: Review blocks PR #426. The host-shell cleanup is useful, but the issue contract requires read-only director/surface sessions and child warded invocations to inherit the current aos .ward ref automatically; this diff does not change that path.
verdict: block confidence: high summary: The patch improves host shell entrypoints, but it does not fix the read-only director surface where the problem was observed. In this current director surface both `WARD_CONFIG_REF` and `AOS_REPO_ROOT` are unset unless the command is manually prefixed. findings: - severity: high file: shell/common.sh line: 16 problem: The implementation only derives `WARD_CONFIG_REF` when `shell/common.sh` is sourced, and the shell entrypoint changes only cover `shell/bashrc` and `shell/zshrc`. impact: The failing surface is a warded read-only director container. That session is not getting either `WARD_CONFIG_REF` or `AOS_REPO_ROOT` from these rc files, so `warded list --json` and `warded engineer ...` still require the manual inline override in the actual surface this issue is about. - severity: medium file: shell/common.sh line: 21 problem: `_siren_aos_repo_root` falls back to `git rev-parse --show-toplevel` before the canonical aos checkout. impact: If `common.sh` is sourced without `AOS_REPO_ROOT` while the current directory is another repo, it can build an agentic-os `WARD_CONFIG_REF` using a non-agentic-os commit SHA. conclusion: Review blocks PR #426. The host-shell cleanup is useful, but the issue contract requires read-only director/surface sessions and child `warded` invocations to inherit the current aos `.ward` ref automatically; this diff does not change that path.
coilyco-ops closed this pull request 2026-07-10 10:13:11 +00:00
coilyco-ops commented 2026-07-10 10:13:31 +00:00
Author
Owner
Copy link

Closing this PR as superseded. Review blocked #426 because it only changed host shell rc files and missed the read-only director container startup path. Replacement PR #430 covers that path, removes the foreign-repo SHA fallback, and has a pass review: #430 (comment)

Closing this PR as superseded. Review blocked #426 because it only changed host shell rc files and missed the read-only director container startup path. Replacement PR #430 covers that path, removes the foreign-repo SHA fallback, and has a pass review: https://forgejo.coilysiren.me/coilyco-flight-deck/agentic-os/pulls/430#issuecomment-29930

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
burndown-2026-06
Backlog burndown June 2026

  
burndown-2026-08
Closed in the 2026-08-26 backlog burn-down. Reopen freely: state:closed label:burndown-2026-08 recovers the whole set.

  
autonomy
async-consult
 A human needs to consult on the issue to upgrade it to headless

  
autonomy
epic
 This issue has many units of sub work - its size makes it meaningfully exclusive with other autonomy types

  
autonomy
headless
 The agent can perform the work on its own

  
autonomy
live-collab
 The agent and the human need to work together in realtime

  
coherence-core
Core review set for the warded control plane coherence milestone. These issues form the release spine; adjacent milestone issues are stretch or supporting work.

  
priority
P0
 priority tier

  
priority
P1
 priority tier

  
priority
P2
 priority tier

  
priority
P3
 priority tier

  
priority
P4
 priority tier

  
qa-fixture
Disposable issue admitted to the bounded Ward QA verification lane.

  
role/advocate
requires work from the Developer Advocate seat

  
role/director
requires work from the Portfolio Director seat

  
role/exec
requires work from the exec role

  
role/frontend
requires work from the Frontend Engineer seat

  
role/gamedev
requires work from the Game Developer seat

  
role/human
requires a person, and specifically not an agent seat

  
role/platform
requires work from the Platform Engineer seat

  
role/qa
requires work from the QA role

  
role/science
requires work from the Applied Scientist seat

  
role/sysadmin
requires work from the Systems Administrator seat

  
state
ambient
 ambient and ephemeral work, held as a maintained document rather than a queue

No labels
burndown-2026-06
burndown-2026-08
autonomy
async-consult
autonomy
epic
autonomy
headless
autonomy
live-collab
coherence-core
priority
P0
priority
P1
priority
P2
priority
P3
priority
P4
qa-fixture
role/advocate
role/director
role/exec
role/frontend
role/gamedev
role/human
role/platform
role/qa
role/science
role/sysadmin
state
ambient
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
coilyco-flight-deck/agentic-os!426
No description provided.