block / block/proto-fleet

chore(sitemap): finalize OMISSION_MODE_LEAVE_IN_PLACE cleanup + E2E for canonical import

Open
#831 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
55
Forks
16
Avg merge
1d 22h
Merged PRs (30d)
87

Description

🤖 Filed by mflesher's AI agent as a follow-up split out of #767.

The canonical resolved-plan refactor of the sitemap CSV import shipped in #808 (single plan feeding preview/validate/token/apply, shared capacity helper). Two items were explicitly deferred out of that PR and are tracked here:

- **Proto-enum cleanup**: remove/finalize the deferred `OMISSION_MODE_LEAVE_IN_PLACE` handling in the omission-mode enum now that the canonical plan applies remove-omitted semantics.
- **E2E coverage**: add end-to-end coverage for the canonical import flow (preview → validate → apply), including the omission-mode paths.

Refs: #767, #808.

Contributor guide

Open the contributing guide

Research direction

Start by reading the canonical sitemap CSV import refactor in #808 and trace its preview, validate, and apply flow. Inspect the omission-mode enum and the deferred OMISSION_MODE_LEAVE_IN_PLACE handling, then add E2E coverage for the canonical flow and its omission-mode paths. Done means the deferred cleanup is finalized and all three stages are covered end to end.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend, testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.