feat: add workspace port inventory support by xergioalex · Pull Request #127 · DailybotHQ/cli

xergioalex · 2026-10-01T03:04:12Z

Summary

Adds reusable workspace service-port allocation and direct access while preserving Caddy focus routing.

Validation

Host Kit ports: 6/6
Host Kit onboarding: 22/22
Host Kit status: 11/11
Bash/Python syntax and git diff checks passed

Scope

Dynamic service ports, workspace port inventory commands, stable satellite exposure, focus proxy updates, and onboarding/Herdr documentation where applicable.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

…nical Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

## Summary Sync the vendored Deep Work Plan skill pack from v6.0.1 to v6.0.2 and re-stamp the harness provenance. ## Change Log - Reinstalled .agents/skills/deepworkplan at v6.0.2 (installed tree was byte-identical to v6.0.1 beforehand, so no local adaptations were lost) - Updated skills-lock.json hash, AGENTS.md DWP standard stamp and the skills catalog version line - Addons already current: AI Diff Reviewer v3.2.2, Dailybot skill 3.23.2 ## Risks - None for CLI users; agent tooling only. .dwp/ plans untouched. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

Brings the launcher and scripts/workspace_ports_display.py to the hub's e893c0c state (herdr-layout tab dedupe, null ports render as dash), so a fresh dbdev workspace checkout no longer shows dev.sh as modified. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Picks up the herdr-layout keep-mode hardening, scoped ports listing and the ports helper's '-' rendering for empty ports in container mode. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

github-actions · 2026-10-01T14:24:05Z

AI review for `ea11826` — ✅ done

View review →

Highest severity: warning

Verifier: 6 checked · 5 verified · 0 downgraded · 0 refuted · 1 unverified · 0 skipped — grok-4.5 (economy), 71s

Strictness gate: ✅ highest severity warning ≤ critical threshold

0 inline comment(s) attached.

Usage: not reported by this provider · 79s

Iteration-Aware Review: gen 6, round 2, policy=first-pass-exhaustive (same_generation, mode=incremental) — 0 surfaced.

github-actions

Code review — 0 finding(s): 0 critical · 0 warning · 0 info

Verification: 0 verified · 0 downgraded · 0 refuted · 0 unverified · 0 skipped — grok-4.5

Check: ✅ passing — strictness block-on-critical: highest severity warning ≤ critical threshold

No findings posted inline.

Summary

No code changed since ea118266, so this round spent no review turns and re-read the 6 outstanding finding(s) with the verifier instead:

scripts/workspace_ports_display.py:152¹ (warning) — verified: AGENTS.md Rule 2 requires type hints on all parameters; _render_urls at line 152 takes untyped records, and the same new file also leaves read_json as bare dict plus untyped to_int/show pa
scripts/workspace_ports_display.py:141² (warning) — verified: Anchor line 141 embeds sock.settimeout(0.2) as a bare literal, and AGENTS.md Rule 12 explicitly requires extracting timeouts/limits into module-level constants. The defect matches the documented rule
scripts/workspace_ports_display.py:91³ (warning) — verified: AGENTS.md Rule 6 requires a failing *_test.py before new behavior; scripts/workspace_ports_display.py is a new file whose render_container encodes non-trivial contracts (4-part port splitting, D
dev.sh:853⁴ (warning) — unverified: verifier ended without a verdict within 4 turns
scripts/workspace_ports_display.py:20⁵ (warning) — verified: to_int at scripts/workspace_ports_display.py:20 is defined as def to_int(value, default: int = 0) -> int, so value lacks a type hint while other annotations are present. AGENTS.md Rule 2 require
scripts/workspace_ports_display.py:27⁶ (warning) — verified: AGENTS.md Rule 2 requires type hints on all parameters; show at line 27 annotates only the return type (-> str) and leaves value bare, same as nearby to_int / _render_urls.

Prior findings

still open scripts/workspace_ports_display.py:152
still open scripts/workspace_ports_display.py:141
still open scripts/workspace_ports_display.py:91
still open dev.sh:853
still open scripts/workspace_ports_display.py:20
still open scripts/workspace_ports_display.py:27

Check status: ✅ passing — strictness block-on-critical, highest severity in effect warning: highest severity warning ≤ critical threshold.

This line is written by the reviewer runtime after the gate ran and matches the check conclusion and the tracking comment. Any recommendation above is the model's advisory opinion, not the gate.

scripts/workspace_ports_display.py:152 is mentioned above but is not a row of the findings table (not posted inline). ↩
scripts/workspace_ports_display.py:141 is mentioned above but is not a row of the findings table (not posted inline). ↩
scripts/workspace_ports_display.py:91 is mentioned above but is not a row of the findings table (not posted inline). ↩
dev.sh:853 is mentioned above but is not a row of the findings table (not posted inline). ↩
scripts/workspace_ports_display.py:20 is mentioned above but is not a row of the findings table (not posted inline). ↩
scripts/workspace_ports_display.py:27 is mentioned above but is not a row of the findings table (not posted inline). ↩

feat(dev): add workspace port inventory command

f3632cd

xergioalex added the Ready Trigger AI code review label Oct 1, 2026

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

github-actions Bot added the pr-reviewed Passed AI code review label Oct 1, 2026

xergioalex and others added 3 commits September 30, 2026 23:13

feat(dev): add focus-aware workspace URLs

060678e

chore(dev): sync launcher dev.sh and ports helper from the hub canonical

bf8e256

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

chore(dev): resync launcher dev.sh and ports helper from the hub cano…

089ce3f

…nical Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

xergioalex added Ready Trigger AI code review and removed Ready Trigger AI code review labels Oct 1, 2026

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

chore(dev): resync launcher dev.sh and ports helper from the hub cano…

bd29af2

…nical Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

xergioalex added Ready Trigger AI code review and removed Ready Trigger AI code review labels Oct 1, 2026

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

chore(dev): resync launcher dev.sh and ports helper from the hub cano…

8d6d4cf

…nical Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

xergioalex added Ready Trigger AI code review and removed Ready Trigger AI code review labels Oct 1, 2026

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

xergioalex added Ready Trigger AI code review and removed Ready Trigger AI code review labels Oct 1, 2026

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

xergioalex and others added 2 commits October 1, 2026 08:15

chore(dev): sync dev.sh and ports helper with the hub canonical

ea11826

Picks up the herdr-layout keep-mode hardening, scoped ports listing and the ports helper's '-' rendering for empty ports in container mode. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

xergioalex removed the Ready Trigger AI code review label Oct 1, 2026

xergioalex added the Ready Trigger AI code review label Oct 1, 2026

This comment has been minimized.

Sign in to view

xergioalex added Ready Trigger AI code review and removed Ready Trigger AI code review labels Oct 1, 2026

This comment was marked as outdated.

Sign in to view

github-actions Bot reviewed Oct 1, 2026

View reviewed changes

oscarmarin621 approved these changes Oct 1, 2026

View reviewed changes

xergioalex merged commit b8a6f74 into main Oct 1, 2026
12 checks passed

Navigation Menu

Repository navigation

feat: add workspace port inventory support - #127

feat: add workspace port inventory support#127
xergioalex merged 9 commits into
mainfrom
feature/dynamic-workspace-ports

xergioalex commented Oct 1, 2026

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

github-actions Bot commented Oct 1, 2026 •

edited

Loading

Uh oh!

github-actions Bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

xergioalex commented Oct 1, 2026

Summary

Validation

Scope

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

Uh oh!

github-actions Bot commented Oct 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

AI review for ea11826 — ✅ done

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Code review — 0 finding(s): 0 critical · 0 warning · 0 info

Summary

Prior findings

Footnotes

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented Oct 1, 2026 •

edited

Loading

AI review for `ea11826` — ✅ done