Skip to content

Commit 87c871a

Browse files
etrclaude
andcommitted
TASK-029: validation fixes — housekeeping (status Done, unworked findings recorded)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent ec82d90 commit 87c871a

3 files changed

Lines changed: 170 additions & 7 deletions

File tree

specs/tasks/M5-routing-lifecycle/TASK-029.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@
88
Collapse synonyms to a single canonical verb per concept, per PRD §3.7.
99

1010
**Action Items:**
11-
- [ ] Rename `webserver::sweet_kill``webserver::stop_and_wait`. Remove the old name.
12-
- [ ] Add `webserver::block_ip(std::string_view ip)` and `webserver::unblock_ip(std::string_view ip)`.
13-
- [ ] Remove `ban_ip`, `unban_ip`, `allow_ip`, `disallow_ip` from the public API. The internal ban list remains; it's just exposed under one name pair.
14-
- [ ] Verify no `// NOLINT(runtime/explicit)` survives on related constructors (covered in TASK-030).
15-
- [ ] Verify `shoutCAST` is preserved as-is (only camelCase exception, per PRD §3.7).
11+
- [x] Rename `webserver::sweet_kill``webserver::stop_and_wait`. Remove the old name.
12+
- [x] Add `webserver::block_ip(std::string_view ip)` and `webserver::unblock_ip(std::string_view ip)`.
13+
- [x] Remove `ban_ip`, `unban_ip`, `allow_ip`, `disallow_ip` from the public API. The internal ban list remains; it's just exposed under one name pair.
14+
- [x] Verify no `// NOLINT(runtime/explicit)` survives on related constructors (covered in TASK-030).
15+
- [x] Verify `shoutCAST` is preserved as-is (only camelCase exception, per PRD §3.7).
1616

1717
**Dependencies:**
1818
- Blocked by: TASK-014
@@ -29,4 +29,4 @@ Collapse synonyms to a single canonical verb per concept, per PRD §3.7.
2929
**Related Requirements:** PRD-NAM-REQ-001, PRD-NAM-REQ-002, PRD-NAM-REQ-005
3030
**Related Decisions:** §3.7, OQ-004, OQ-005
3131

32-
**Status:** Not Started
32+
**Status:** Done

specs/tasks/_index.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ Nominally: **13 sequential tasks**, each S–XL. Most other tasks parallelize of
111111
| TASK-026 | Generic `webserver::route(method, path, handler)` | M4 | Done | TASK-005, TASK-025 |
112112
| TASK-027 | 3-tier route table with LRU cache | M5 | Done | TASK-005, TASK-014, TASK-021, TASK-024, TASK-025, TASK-026 |
113113
| TASK-028 | Routing-semantics regression gate | M5 | Done | TASK-027 |
114-
| TASK-029 | Naming consistency — `stop_and_wait`, `block_ip`/`unblock_ip` | M5 | Not Started | TASK-014 |
114+
| TASK-029 | Naming consistency — `stop_and_wait`, `block_ip`/`unblock_ip` | M5 | Done | TASK-014 |
115115
| TASK-030 | `_handler` suffix renames + `explicit` constructor | M5 | Not Started | TASK-014 |
116116
| TASK-031 | Handler error-propagation contract (DR-009) | M5 | Not Started | TASK-027, TASK-030 |
117117
| TASK-032 | Thread-safety contract stress test (DR-008) | M5 | Not Started | TASK-027, TASK-031 |

0 commit comments

Comments
 (0)