Skip to content

Commit 7a04650

Browse files
committed
code-review-fix checklist sync completed task flags
1 parent b84487a commit 7a04650

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

  • openspec/changes/code-review-fix

openspec/changes/code-review-fix/tasks.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
## 1. Mark Creation And Mutation Safety
22

3-
- [ ] 1.1 Refactor `new_mark` flow to use a single ORM object path and one commit for successful requests. `Complexity: Low`
3+
- [x] 1.1 Refactor `new_mark` flow to use a single ORM object path and one commit for successful requests. `Complexity: Low`
44
- [x] 1.2 Add request-path tests for mark creation with title and without optional fields to prevent regression. `Complexity: Low`
55
- [x] 1.3 Convert state-changing mark routes from GET to POST/DELETE handlers and update route wiring. `Complexity: Middle`
66
- [x] 1.4 Update templates/forms for mutation endpoints to submit CSRF-protected requests. `Complexity: Low`
7-
- [ ] 1.5 Add regression tests proving GET mutation requests are rejected and CSRF validation is enforced. `Complexity: Middle`
7+
- [x] 1.5 Add regression tests proving GET mutation requests are rejected and CSRF validation is enforced. `Complexity: Middle`
88

99
## 2. HTML Safety And URL Fetch Controls
1010

1111
- [x] 2.1 Implement HTML sanitization/escaping policy for externally sourced page content before rendering. `Complexity: High`
1212
- [x] 2.2 Update HTML display templates to ensure unsafe markup is never executed in user context. `Complexity: Middle`
1313
- [x] 2.3 Implement URL scheme and resolved-address validation to allow only public `http/https` targets. `Complexity: High`
14-
- [ ] 2.4 Add fetch-path tests for blocked loopback/private/link-local targets and allowed public targets. `Complexity: Middle`
14+
- [x] 2.4 Add fetch-path tests for blocked loopback/private/link-local targets and allowed public targets. `Complexity: Middle`
1515
- [x] 2.5 Add security regression tests for stored XSS neutralization behavior. `Complexity: Middle`
1616

1717
## 3. Configuration And Import Status Isolation

0 commit comments

Comments
 (0)