Skip to content

Commit beaaf80

Browse files
etrclaude
andcommitted
TASK-023: housekeeping — mark Done, record unworked review issues
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 7f2e86f commit beaaf80

3 files changed

Lines changed: 136 additions & 7 deletions

File tree

specs/tasks/M4-handlers/TASK-023.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@
88
Replace the raw-pointer `register_resource` overload with `unique_ptr` and `shared_ptr` overloads so ownership is explicit at the call site.
99

1010
**Action Items:**
11-
- [ ] Add `void register_resource(const std::string& path, std::unique_ptr<http_resource> resource);` (move-in ownership; library internally upgrades to `shared_ptr` for thread-safe lookup).
12-
- [ ] Add `void register_resource(const std::string& path, std::shared_ptr<http_resource> resource);` (caller retains a reference).
13-
- [ ] Remove the raw-pointer overload `register_resource(string, http_resource*, bool)`.
14-
- [ ] Update internal route-table entries to hold `std::shared_ptr<http_resource>` (`route_entry`'s variant per §4.7).
15-
- [ ] Update examples and tests to use the new ownership model.
11+
- [x] Add `void register_resource(const std::string& path, std::unique_ptr<http_resource> resource);` (move-in ownership; library internally upgrades to `shared_ptr` for thread-safe lookup).
12+
- [x] Add `void register_resource(const std::string& path, std::shared_ptr<http_resource> resource);` (caller retains a reference).
13+
- [x] Remove the raw-pointer overload `register_resource(string, http_resource*, bool)`.
14+
- [x] Update internal route-table entries to hold `std::shared_ptr<http_resource>` (`route_entry`'s variant per §4.7).
15+
- [x] Update examples and tests to use the new ownership model.
1616

1717
**Dependencies:**
1818
- Blocked by: TASK-014
@@ -28,4 +28,4 @@ Replace the raw-pointer `register_resource` overload with `unique_ptr` and `shar
2828
**Related Requirements:** PRD-HDL-REQ-003, PRD-HDL-REQ-005
2929
**Related Decisions:** §4.4, §4.7
3030

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

specs/tasks/_index.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ Nominally: **13 sequential tasks**, each S–XL. Most other tasks parallelize of
105105
| TASK-020 | Final public-header backend-include sweep | M3 | Done | TASK-014, TASK-015, TASK-019 |
106106
| TASK-021 | `http_resource` allow-mask via `method_set` | M4 | Done | TASK-005 |
107107
| TASK-022 | Snake_case `render_*` overrides on `http_resource` | M4 | Done | TASK-021 |
108-
| TASK-023 | Smart-pointer `register_resource` overloads | M4 | Not Started | TASK-014 |
108+
| TASK-023 | Smart-pointer `register_resource` overloads | M4 | Done | TASK-014 |
109109
| TASK-024 | `register_path` and `register_prefix` (replace `bool family`) | M4 | Not Started | TASK-023 |
110110
| TASK-025 | Lambda handler entry points `on_*` | M4 | Not Started | TASK-005, TASK-009, TASK-014 |
111111
| TASK-026 | Generic `webserver::route(method, path, handler)` | M4 | Not Started | TASK-005, TASK-025 |

0 commit comments

Comments
 (0)