| 1 | # Current |
| 2 | |
| 3 | > Keep this file short. One active step, one ordered backlog. Completed work moves to |
| 4 | > [progress.md](progress.md). If this file starts reading like a changelog, it has |
| 5 | > drifted — that's exactly what went wrong last time. |
| 6 | |
2eb8681docs: put git next, plan the repo model24d | 7 | ## Active: Milestone 3 — Repo model |
| 8 | |
| 9 | **Goal:** repositories exist as records and as bare git repos on disk, and they appear |
| 10 | on the profile. No git protocol yet — that is milestone 4. This milestone fills the |
| 11 | Repositories section and gets the storage layout right before anything serves it. |
| 12 | |
| 13 | **Out of scope:** clone, push, browsing a tree, README rendering, forks, stars. |
| 14 | Deleting a repo — worth having, but it makes the filesystem/database consistency |
| 15 | problem twice as interesting, so not in the first pass. |
| 16 | |
| 17 | ### Steps |
| 18 | |
c6d74a8docs: record the repo model decisions24d | 19 | - [x] Domain: `RepoId`, `RepoName`, `Visibility` (Public/Private), `Repository` |
c5b3ff5feat: repository persistence24d | 20 | - [x] Domain: `RepoRepository` port — `find_by_id`, `find_by_org_and_name`, |
2eb8681docs: put git next, plan the repo model24d | 21 | `list_by_org`, `save` |
c5b3ff5feat: repository persistence24d | 22 | - [x] Infrastructure: in-memory + SQLite implementations, migration |
02eb2e4feat: GitStorage port and DiskGitStorage24d | 23 | - [x] Application: `GitStorage` port — `init_bare`, `remove`, `repo_path` |
| 24 | - [x] Infrastructure: `DiskGitStorage`, shelling out to `git init --bare` |
11e7a39feat: create_repo use case24d | 25 | - [x] Application: `create_repo` use case — owner only, validates, creates record and |
2eb8681docs: put git next, plan the repo model24d | 26 | bare repo |
285f5fdfeat: create and view repositories through the browser24d | 27 | - [x] Application: `view_repo` read model — visibility-aware |
| 28 | - [x] Web: `/{handle}/repos/new` form, `/{handle}/repos/{name}` page |
| 29 | - [ ] Application: `list_repos` + the profile's Repositories section listing what the |
| 30 | viewer may see |
2eb8681docs: put git next, plan the repo model24d | 31 | - [ ] `/api/users/{handle}/repos` |
| 32 | |
| 33 | ### Done when |
| 34 | |
2eb8681docs: put git next, plan the repo model24d | 35 | The owner creates a repo through the UI, a bare repo appears at |
| 36 | `{data_dir}/{handle}/{name}.git`, and it is listed on the profile. A private repo is |
| 37 | invisible to a signed-out visitor. `git clone` does **not** work yet — that is |
| 38 | milestone 4. |
| 39 | |
c6d74a8docs: record the repo model decisions24d | 40 | ### Settled |
2eb8681docs: put git next, plan the repo model24d | 41 | |
285f5fdfeat: create and view repositories through the browser24d | 42 | - **`list_repos` was split from `view_repo`** and moved to the listing step. `view_repo` |
| 43 | had a consumer immediately; `list_repos` would have been a third read model with no |
| 44 | caller, which is what the previous three steps already were. |
| 45 | - **An invisible private repo is `None`, not `Forbidden`.** `view_repo` answers the same |
| 46 | way for "does not exist" and "not allowed to see", and the page 404s identically. A |
| 47 | 403 would confirm the repository exists and leak its name. |
| 48 | - **Private repos are visible to any member**, not only the owner — seeing is weaker |
| 49 | than changing, matching attempt #2's clone rule. |
| 50 | |
c6d74a8docs: record the repo model decisions24d | 51 | - **Repo name rules:** `OrgName`'s, plus `.` and `_` for names like `.github` and |
| 52 | `foo.js`. Lowercased, max 100. Also rejects a name of nothing but dots and any name |
| 53 | ending `.git` — the first is traversal, the second would live at `foo.git.git`. |
| 54 | - **Reserved repo names:** `import`, `new`, `search`. Only names directly under |
| 55 | `/{handle}/repos/` can collide. |
| 56 | - **Visibility defaults to public**, matching a portfolio-first product. |
| 57 | - **Repositories carry an optional description**, capped at 300 characters — a sentence |
02eb2e4feat: GitStorage port and DiskGitStorage24d | 58 | for the profile listing, not a README. Kept deliberately: portfolio-first is the |
| 59 | tie-break, and this milestone's own "Done when" puts repositories on the profile, so |
| 60 | the consumer is inside the milestone rather than hypothetical. The per-repo analogue |
| 61 | of `Organization::bio`. |
c5b3ff5feat: repository persistence24d | 62 | - **`list_by_org` returns every repository regardless of visibility.** Filtering is an |
| 63 | authorization decision and belongs to the use case, so the page and `/api` cannot end |
| 64 | up applying different rules. The cost is that a private repo is briefly in memory |
| 65 | before being filtered, which is fine in-process. |
3954b45docs: record milestone 2 phase 125d | 66 | |
be4fac5docs: correct the Topcoat guidance in CLAUDE.md24d | 67 | ### Open |
| 68 | |
02eb2e4feat: GitStorage port and DiskGitStorage24d | 69 | Nothing open. `GitStorage`'s shape and how git is invoked are recorded in |
| 70 | [0006](decisions/0006-git-binary-behind-narrow-ports.md). |
be4fac5docs: correct the Topcoat guidance in CLAUDE.md24d | 71 | |
bd48b4bdocs: serve git over smart HTTP, reorder roadmap portfolio-first1mo | 72 | ### Watch for |
d7b99d9docs: record milestone 0 progress and routing findings1mo | 73 | |
285f5fdfeat: create and view repositories through the browser24d | 74 | - **`redirect()` is a 307 and re-POSTs.** Post/redirect/get needs a 303. `see_other()` |
| 75 | is the right status but is a *response* type, and `#[page]` must return a view so the |
| 76 | layout can wrap a failure re-render. The way to get both from one handler is a |
| 77 | `StatusCode` and a `Location` pair inside `view!` — wrapped as |
| 78 | `context::location`. This shipped broken in Milestone 2's settings form and was |
| 79 | caught by browser verification, not by any test. |
| 80 | |
11e7a39feat: create_repo use case24d | 81 | - **An orphaned directory is indistinguishable from a duplicate to the visitor.** |
| 82 | `create_repo` maps `GitStorageError::AlreadyExists` to "that name is taken", which is |
| 83 | true from outside but hides the inconsistency from the operator. There is no logging |
| 84 | story yet for it to surface in. The durable fix is the reconciliation sweep in |
| 85 | [architecture.md](architecture.md#db-plus-filesystem-writes). |
| 86 | - **The duplicate check races.** Two concurrent creates of the same name can both pass |
| 87 | `find_by_org_and_name`; the loser is then stopped by `init_bare` or, failing that, by |
| 88 | the `unique (org_id, name)` constraint — which surfaces as an opaque storage error |
| 89 | rather than "name taken". Correct, just ugly, and single-user for now. |
| 90 | |
2eb8681docs: put git next, plan the repo model24d | 91 | - **The database and the filesystem cannot share a transaction.** Creating a repo |
| 92 | writes a row and a directory. Neither previous attempt solved this properly — see |
| 93 | [architecture.md](architecture.md#db-plus-filesystem-writes). A compensating delete is |
| 94 | good enough to ship, but write down that an orphaned directory is possible if the |
| 95 | process dies between the two, rather than rediscovering it. |
| 96 | - **Path traversal.** `{data_dir}/{handle}/{name}.git` is built from user input. A name |
| 97 | containing `..` or `/` must be impossible before it reaches the filesystem, and |
| 98 | `RepoName` is the place to make it impossible rather than sanitising at the call site. |
| 99 | - **Visibility is an authorization decision**, so it belongs in the use case. A private |
| 100 | repo must be absent from listings, not merely unlinked — and `/api` must agree with |
| 101 | the page. |
02eb2e4feat: GitStorage port and DiskGitStorage24d | 102 | - **`git` is a dependency of the test suite too**, not only of the runtime — |
| 103 | `DiskGitStorage`'s tests run real `git init`. A machine without `git` fails |
| 104 | `cargo test`, not just the app. |
| 105 | - **A handle rename is a directory move.** The layout is keyed by handle for |
| 106 | legibility, so whenever renaming arrives it has to move the tree; it cannot be a row |
| 107 | update. Nothing renames handles today. |
| 108 | - **Bare repos created on macOS carry `ignorecase = true`** in their config, because |
| 109 | git probes the filesystem at init. Correct where it was created, wrong if the data |
| 110 | directory is ever moved to Linux. A migration gotcha, not a bug. |
| 111 | - **An orphaned directory blocks re-creating that name.** `init_bare` refuses rather |
| 112 | than adopting what is already there, and repo deletion is out of scope this |
| 113 | milestone, so clearing one is a manual `rm` for now. |
076dbc9docs: close milestone 1, open milestone 21mo | 114 | |
f0444b7docs: plan milestone 2 in two phases1mo | 115 | ### Carried over — small, unblocked |
076dbc9docs: close milestone 1, open milestone 21mo | 116 | |
2eb8681docs: put git next, plan the repo model24d | 117 | - **Fonts are not loaded.** The theme names Geist and IBM Plex Mono; both fall back |
| 118 | today. Topcoat's `font-fontsource` feature handles it. |
| 119 | - **Light mode is untested.** The palette defines it; nobody has looked at it. |
f0444b7docs: plan milestone 2 in two phases1mo | 120 | - **No rate limiting** on `/auth/login` or `/auth/setup`. |
076dbc9docs: close milestone 1, open milestone 21mo | 121 | - **`sweep_expired` is never called**, so expired session rows accumulate. Expiry is |
| 122 | enforced on read, so this is tidiness, not a hole. |
2eb8681docs: put git next, plan the repo model24d | 123 | - **CSRF.** `SameSite=Lax` covers the common case. Forms now exist, so this is decidable |
| 124 | rather than hypothetical. |
| 125 | |
| 126 | ## Backlog |
| 127 | |
| 128 | Ordered. Pull from the top. |
| 129 | |
2eb8681docs: put git next, plan the repo model24d | 130 | 1. **Milestone 4 — Git over HTTP.** `git http-backend` subprocess, PATs over HTTP |
| 131 | Basic. See [0001](decisions/0001-git-over-http-not-ssh.md). The `body_limit` cap will |
| 132 | reject large pushes until raised. |
02eb2e4feat: GitStorage port and DiskGitStorage24d | 133 | 2. **Milestone 5 — Repo browsing.** Tree, blob, commit log. **Start with domain value |
| 134 | objects** — `ObjectId`, `RefName`, `TreeEntry` — before any adapter. A query port |
| 135 | returning `String`s is an anaemic pass-through that pushes validation into the page. |
| 136 | Also the point to measure fork/exec cost per page view, and to reconsider `gix` for |
| 137 | the read path ([0006](decisions/0006-git-binary-behind-narrow-ports.md)). |
2eb8681docs: put git next, plan the repo model24d | 138 | 3. **Milestone 6 — Writing.** Posts, markdown, `/{handle}/posts/{slug}`. Still open |
| 139 | whether writing or projects/showcases is the better first portfolio feature. |
| 140 | |
| 141 | ## Open questions |
| 142 | |
bd48b4bdocs: serve git over smart HTTP, reorder roadmap portfolio-first1mo | 143 | - **Topcoat is early** (v0.5.0, first released 2026-07-22, breaking changes expected |
| 144 | by its own authors). Expect churn that isn't feature work. |
ca76e1bdocs: fix milestone cross-references after the reorder24d | 145 | - Body size limits will reject large pushes at Milestone 4 — `topcoat-router` has a |
bd48b4bdocs: serve git over smart HTTP, reorder roadmap portfolio-first1mo | 146 | `body_limit` layer that needs raising on the git routes. Recorded here because it |
| 147 | will surface as a confusing failure rather than a clear one. |
| 148 | - Topcoat ships Tailwind without Node, which reopens the design system attempt #1 |
| 149 | dropped purely to avoid an npm build step — see [ui.md](ui.md). |
| 150 | |
| 151 | ## Routing findings (Milestone 0) |
| 152 | |
| 153 | - **Topcoat 0.5 requires rustc ≥ 1.95.** On an older toolchain `cargo add topcoat` |
| 154 | silently resolves to an empty `topcoat v0.0.0` placeholder instead of failing. Local |
| 155 | stable is now 1.97.1. |
| 156 | - `Router::builder().discover()` collects `#[page]`-annotated items **at link time**, |
| 157 | so pages can live in any module. Layering is our choice, not the framework's. |
| 158 | - `module_router!` derives each URL from the module tree rather than a path string. |
aaefaabfeat: root handles, grouped routes, reserved-handle denylist1mo | 159 | Still deferred. Application routes now group cleanly (`auth/login`, `api/me`), but |
| 160 | handles sit at the root ([0004](decisions/0004-root-handles-grouped-routes.md)), so a |
| 161 | parameterised root segment still has to coexist with static ones. Worth checking how |
| 162 | `module_router!` handles that before committing to it. |
bd48b4bdocs: serve git over smart HTTP, reorder roadmap portfolio-first1mo | 163 | - Path and query params are read from `Cx` via `path_param!` / `#[query_params]`, not |
| 164 | injected as handler arguments. Parses are memoized per request. |
| 165 | - Layouts wrap by path prefix and nest outermost-first, and a layout can catch a page's |
| 166 | `NotFoundError` to render a branded 404. |
| 167 | - `HOST` / `PORT` configure the bind address, so `STEID_LISTEN_ADDR` is gone. |
| 168 | - `Body` is a boxed `http_body::Body` used for both requests and responses, with |
| 169 | `into_data_stream()` to read and `Body::new()` to wrap a stream — pack data can |
ca76e1bdocs: fix milestone cross-references after the reorder24d | 170 | stream both directions without buffering. This is what makes Milestone 4 viable. |