docs(network): tailscale ACL — fill 5 TODO comments #65

Closed
opened 2026-05-05 01:07:01 +02:00 by glm · 1 comment

network/tailscale-acl.hujson has 5 TODO comments that need real values before operator can paste into Tailscale admin.

Spec sources

  • network/tailscale-acl.hujson (237 lines, 5 TODOs)
  • PLATFORM_CHARTER.md §3 + §11
  • modules/ (for /metrics port enumeration)

Extracted context

Line 55: TODO confirm exact operator identity string
Line 117: TODO enumerate all /metrics ports per module manifest
Line 186: TODO validate ACL test extensions
Line 216: TODO enable after operator generates lock keys
Line 6: Status SEED with explicit TODOs

What done looks like

  • group:operator has confirmed identity string
  • /metrics ports enumerated from module manifests
  • All TODOs resolved or documented as owner-attention

Risk: risk/exposure, owner-attention

Trace: Issue #59 item 6, decomposed by glm 2026-05-05, parent #59

network/tailscale-acl.hujson has 5 TODO comments that need real values before operator can paste into Tailscale admin. ## Spec sources - network/tailscale-acl.hujson (237 lines, 5 TODOs) - PLATFORM_CHARTER.md §3 + §11 - modules/ (for /metrics port enumeration) ## Extracted context > Line 55: TODO confirm exact operator identity string > Line 117: TODO enumerate all /metrics ports per module manifest > Line 186: TODO validate ACL test extensions > Line 216: TODO enable after operator generates lock keys > Line 6: Status SEED with explicit TODOs ## What done looks like - [ ] group:operator has confirmed identity string - [ ] /metrics ports enumerated from module manifests - [ ] All TODOs resolved or documented as owner-attention ## Risk: risk/exposure, owner-attention ## Trace: Issue #59 item 6, decomposed by glm 2026-05-05, parent #59
Collaborator

Ready for agent

Execution protocol: state/agent-execution-template.md (PR #69 → main).

Specifics for this issue:

  • Recommended actor: claude (charter-adjacent work; needs cross-reading of modules/ for /metrics ports + PLATFORM_CHARTER §3, §11) — glm acceptable but should expect more "STOP and ask operator" moments
  • Spec sources whitelist:
    • network/tailscale-acl.hujson (237 lines, 5 TODOs at lines 6, 55, 117, 186, 216)
    • PLATFORM_CHARTER.md §3 (review-by-impact) + §11 (sacred paths) + any §exposure section
    • modules/*/module.yaml (filter for metrics_port or prometheus_port fields — needed for TODO at line 117)
    • state/reports/STATE_OF_PLATFORM_2026-05-03.md (for context on tailnet-only exposure stance)
  • Allowed touched paths:
    • network/tailscale-acl.hujson (modify — fill TODOs OR document each as owner-attention follow-up)
    • NO other paths — this PR is comment-fill only, not policy change
  • Self-verification commands:
    • python3 -c "import json; json.loads(open('network/tailscale-acl.hujson').read())" ← will likely fail (HUJSON has comments); use HUJSON parser if available, OR strip comments + verify
    • grep -c "TODO" network/tailscale-acl.hujson ← should be 0 after, OR each remaining TODO documented in PR body as deferred to operator
  • Escape hatches (specific):
    • If TODO at line 55 (operator identity string) requires operator decision, leave as TODO + label issue with owner-attention (already has it)
    • If TODO at line 216 (lock keys) requires operator UI work, leave as TODO + document
    • If /metrics port enumeration finds modules with no port declared, comment on this issue listing them — do NOT make up ports

Per AGENTS.md sacred-paths discipline: network/tailscale-acl.hujson is comment-heavy SEED, never auto-applied to live Tailscale. PR is doc-only; operator manually pastes when ready.

PR title format: docs(network): tailscale ACL — resolve 5 TODOs to actionable values or owner-attention deferralsCloses #65.

## Ready for agent Execution protocol: **`state/agent-execution-template.md`** (PR #69 → main). **Specifics for this issue:** - **Recommended actor**: `claude` (charter-adjacent work; needs cross-reading of modules/ for /metrics ports + PLATFORM_CHARTER §3, §11) — `glm` acceptable but should expect more "STOP and ask operator" moments - **Spec sources whitelist**: - `network/tailscale-acl.hujson` (237 lines, 5 TODOs at lines 6, 55, 117, 186, 216) - `PLATFORM_CHARTER.md` §3 (review-by-impact) + §11 (sacred paths) + any §exposure section - `modules/*/module.yaml` (filter for `metrics_port` or `prometheus_port` fields — needed for TODO at line 117) - `state/reports/STATE_OF_PLATFORM_2026-05-03.md` (for context on tailnet-only exposure stance) - **Allowed touched paths**: - `network/tailscale-acl.hujson` (modify — fill TODOs OR document each as `owner-attention` follow-up) - **NO other paths** — this PR is comment-fill only, not policy change - **Self-verification commands**: - `python3 -c "import json; json.loads(open('network/tailscale-acl.hujson').read())"` ← will likely fail (HUJSON has comments); use HUJSON parser if available, OR strip comments + verify - `grep -c "TODO" network/tailscale-acl.hujson` ← should be 0 after, OR each remaining TODO documented in PR body as deferred to operator - **Escape hatches (specific)**: - If TODO at line 55 (operator identity string) requires operator decision, leave as TODO + label issue with `owner-attention` (already has it) - If TODO at line 216 (lock keys) requires operator UI work, leave as TODO + document - If `/metrics` port enumeration finds modules with no port declared, comment on this issue listing them — do NOT make up ports **Per AGENTS.md sacred-paths discipline**: `network/tailscale-acl.hujson` is comment-heavy SEED, never auto-applied to live Tailscale. PR is doc-only; operator manually pastes when ready. PR title format: `docs(network): tailscale ACL — resolve 5 TODOs to actionable values or owner-attention deferrals` — `Closes #65`.
Sign in to join this conversation.
No labels
W6d-automerge-calibration
agent/claude-code
agent/codex
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
automerge-candidate
class/security-sensitive
cutover-gate
dependency/blocked
dependency/blocks-others
dependency/cross-repo
dependency/needs-confirmation
domain:agents
domain:ci
domain:docs
domain:forgejo
domain:infra
domain:memory
domain:runtime
domain:signal
domain:ux
flow/architecture
flow/blocked
flow/deployed
flow/done
flow/implementation
flow/intake
flow/maintained
flow/observed
flow/ready
flow/refining
flow/retired
flow/review
iterating
judge/codex-candidate
judge/hermes-candidate
judge/low-confidence
judge/needs-refinement
judge/operator-needed
judge/p0
judge/p1
judge/p2
judge/p3
judge/park
judge/patchwarden-candidate
judge/stale-priority
kind/adr
kind/bug
kind/chore
kind/feature
kind/infra
kind/ops
kind/refactor
kind/research
large-impact
merge/auto
merge/manual
merge/manual-dependency-conflict
merge/manual-failing-tests
merge/manual-merge-conflict
merge/manual-missing-review
merge/manual-operator-preference
merge/manual-red-zone
merge/manual-security-sensitive
merge/manual-unclear-scope
merge/manual-unknown
meta
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
needs-operator-decision
needs-triage
not-ready
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
operator-emotional
owner-attention
phase/02
phase/03
priority:p0
priority:p1
priority:p2
priority:p3
proposed
ready-for-agent
ready-for-operator
recovery
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
risk/exposure
risk/process
risk/product
risk/runtime
safety:external-write
safety:no-prod-mutation
safety:prod-impact
safety:secret-touch
size/large
size/medium
size/small
size/tiny
size/unknown
source/adr
source/agent-generated
source/manual
source/operator-chat
source/voice-note
status:blocked
status:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:operator-needed
status:parked
tier/full
tier/lite
tier/stacked
tier:0-platform-substrate
tier:1-iskra-value-layer
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
No milestone
No project
No assignees
2 participants
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
pdurlej/platform#65
No description provided.