| 1 | # Progress |
| 2 | |
| 3 | ## This attempt (#3, Topcoat) |
| 4 | |
c50d9adfeat: profile settings with flash feedback25d | 5 | 117 tests. Active milestone in [current.md](current.md). |
88583f2docs: bring tracking docs up to date with milestone 11mo | 6 | |
| 7 | ### Milestone 0 — Skeleton · done |
| 8 | |
| 9 | Topcoat 0.5 app serving pages, `AppConfig` from `STEID_*` env, SQLite pool in app |
| 10 | context. Split into a library plus a thin binary — the domain layer had no consumers |
| 11 | yet and read as ~30 dead-code warnings in a bare binary, and it unlocks `tests/`. |
| 12 | Topcoat's link-time page discovery works from a library; that was checked, not assumed. |
| 13 | |
| 14 | **Requires rustc ≥ 1.95.** On older toolchains `cargo add topcoat` silently resolves to |
| 15 | an empty `topcoat v0.0.0` placeholder instead of failing. |
| 16 | |
076dbc9docs: close milestone 1, open milestone 21mo | 17 | ### Milestone 1 — Identity, thin · done |
88583f2docs: bring tracking docs up to date with milestone 11mo | 18 | |
| 19 | **Domain.** Typed IDs, `Email`, `PasswordHash`, `OrgName`, `Organization`, `User`, |
| 20 | `Membership`, `Role`, `Actor`, `Session`, `SetupToken`, `DomainError`. Value objects |
| 21 | pair `new()` (validates) with `from_trusted()` (skips, for rows already validated). |
| 22 | |
| 23 | **Application.** `claim_instance`, `login`, `resolve_actor`, `record_session`, |
| 24 | `end_session`, `sweep_expired`. `PasswordHasher` port with Argon2 and a stub. |
| 25 | |
| 26 | **Infrastructure.** Migrations for orgs, users, memberships, sessions. In-memory and |
076dbc9docs: close milestone 1, open milestone 21mo | 27 | SQLite implementations of every port. Root layout, `/setup`, `/login`, `/logout`, home, |
| 28 | and `/api/me`. |
| 29 | |
| 30 | `describe_identity` is the first use case with two consumers — the home page and |
| 31 | `/api/me` both read through it. Until that existed, "the application layer is |
| 32 | transport-neutral" was an assertion with one caller behind it. |
88583f2docs: bring tracking docs up to date with milestone 11mo | 33 | |
| 34 | **Verified in a browser and by curl:** wrong token refused with nothing written; |
| 35 | correct token creates org + user + owner membership and signs the owner in; session |
| 36 | authenticates; logout clears cookie and row; wrong password bounces; right one signs |
| 37 | in; re-claiming a claimed instance is refused. |
| 38 | |
| 39 | #### Decisions worth remembering |
| 40 | |
| 41 | - **`Actor` is an enum with an explicit `Anonymous`**, not `Option<UserId>`. Attempt #2 |
| 42 | used a placeholder `UserId("ssh-anonymous")` and it became a security hole. A variant |
| 43 | can't be forgotten the way a sentinel can. |
| 44 | - **`login` verifies a dummy hash when no user matched.** Returning early on the |
| 45 | unknown-email path makes it measurably faster and leaks which addresses have |
| 46 | accounts. A test pins that the dummy stays parseable — if it stops being, `verify` |
| 47 | bails early and the defence dies silently. |
| 48 | - **`SetupToken` compares in constant time.** An early-return comparison leaks how much |
| 49 | of the token is right, which recovers it a character at a time. |
| 50 | - **The setup token is only in app context while unclaimed**, so a claimed instance has |
| 51 | nothing for a claim attempt to match. |
| 52 | - **SQLite ignores foreign keys unless asked**, per connection. `foreign_keys(true)` |
| 53 | plus a test that a user pointing at a missing org is refused. |
| 54 | - **`sqlx migrate add` stamps versions to the second** — three calls in one second |
| 55 | collide, leaving apply order ambiguous between tables that reference each other. |
| 56 | - **An unparseable role surfaces as an error**, never as "no membership". The latter |
| 57 | silently downgrades an owner to no access. |
| 58 | - **`Secure` session cookies over plain-HTTP localhost fail silently.** See |
| 59 | [runbook.md](runbook.md#steid_insecure_cookies--development-only). This one actually |
| 60 | bit, and it looked exactly like broken auth logic. |
| 61 | |
c50d9adfeat: profile settings with flash feedback25d | 62 | ### Milestone 2 — Profile page · done |
3954b45docs: record milestone 2 phase 125d | 63 | |
| 64 | `/{handle}` is the real profile page: label, handle, bio, and the section frame for |
| 65 | repositories, writing, and projects. Public, renders signed out, 404s on an unknown |
| 66 | handle, and resolves regardless of casing. `/` forwards a signed-in owner to their own |
| 67 | profile. `/api/users/{handle}` serves the same read model as JSON. |
| 68 | |
| 69 | URLs settled as root handles with grouped application routes |
| 70 | ([0004](decisions/0004-root-handles-grouped-routes.md)), superseding |
| 71 | [0003](decisions/0003-scoped-urls.md) the same day. A twenty-word reserved list in |
| 72 | `OrgName::new` keeps handles from shadowing routes. |
| 73 | |
| 74 | #### Decisions worth remembering |
| 75 | |
| 76 | - **`PublicProfile` has no email field, deliberately.** `Identity` does, and `/api/me` |
| 77 | returns it, because that endpoint describes the caller to themselves. Giving the type |
| 78 | that reaches the page nowhere to put an email makes the leak impossible rather than |
| 79 | merely avoided. |
| 80 | - **`viewer_is_owner` is decided in the use case**, so the web form and `/api` cannot |
| 81 | disagree about who may edit. Tested for a signed-in stranger and a non-owner member — |
| 82 | "signed in" quietly becoming "allowed" is the usual failure. |
| 83 | - **`Organization::update_profile` clears on blank input** rather than storing |
| 84 | whitespace, so cleared and never-set are one state and the page renders one case. A |
| 85 | rejected edit applies nothing. |
| 86 | - **Bio length counts characters, not bytes.** A byte check would reject a bio of |
| 87 | accented text well under the limit. |
| 88 | - **`path_param` is an attribute macro in 0.5**, not function-like. The vendored crate |
| 89 | is the authority for the pinned version, not the docs on `main`. |
c50d9adfeat: profile settings with flash feedback25d | 90 | - **Components are invoked bare inside `view!`**, and `if`/`match`/`for`/`let` are |
| 91 | native to the macro. |
| 92 | - **`#[query_params]` needs `error = …`** to work with `?`; otherwise the error borrows |
| 93 | from `cx` and escapes the handler. |
| 94 | - **Forms re-render on failure and redirect on success.** Redirecting after a |
| 95 | validation error throws away what was typed and hides the reason. |
| 96 | - **Styling is Tailwind via Topcoat's build script**, with registry components copied |
| 97 | in rather than depended on ([0005](decisions/0005-tailwind-and-copied-components.md)). |
| 98 | Components reference theme tokens, never raw colours. |
3954b45docs: record milestone 2 phase 125d | 99 | |
| 100 | --- |
| 101 | |
| 102 | ## Reference: what attempt #2 proved |
| 103 | |
| 104 | Not this repo's progress. This is a catalogue of what was built and **verified working** |
| 105 | in `steid-backup-2026-07-31`, so the rebuild can crib rather than rediscover. |
| 106 | |
| 107 | Final state: single crate, ~4,200 LOC, 60 passing tests, five milestones. |
| 108 | |
| 109 | ### Identity |
| 110 | |
| 111 | Domain model (User, Organization, Membership, Actor, Role), `Email` and |
| 112 | `PasswordHash` value objects, typed IDs, `DomainError`, four repository ports with |
| 113 | in-memory and SQLite implementations each. `RegistrationPolicy` (Personal / Invite / |
| 114 | Open) driving which routes exist. Argon2 hashing behind a `PasswordHasher` port with a |
| 115 | stub for tests. Use cases: `bootstrap_owner`, `register_user`, `login`, `create_invite`. |
| 116 | Signed-cookie sessions via an `AuthUser` extractor. |
| 117 | |
| 118 | **Gotcha:** organizations must be saved before users — the FK runs that direction. |
| 119 | Both `bootstrap_owner` and `register_user` had to be fixed for this. |
| 120 | |
| 121 | ### Repo model |
| 122 | |
| 123 | `Repository` entity, `RepoId`, `Visibility` (Public/Private), `RepoRepository` port, |
| 124 | migration `006_create_repositories.sql`. `create_repo` use case validates the name, |
| 125 | rejects duplicates, and initialises the bare repo on disk in the same call. Bare repos |
| 126 | live at `{data_dir}/{org}/{repo}.git`, `data_dir` defaulting to `./data`. Repos are |
| 127 | created empty, no initial commit, like GitHub. |
| 128 | |
| 129 | ### Git over SSH |
| 130 | |
| 131 | `GitStorage` port (`init_bare`, `repo_path`) with `DiskGitStorage` shelling out to |
| 132 | `git init --bare`. `GitProtocolServer` port (`upload_pack`, `receive_pack`) with |
| 133 | `GitBinary` spawning `git upload-pack` / `git receive-pack` via |
| 134 | `tokio::process::Command` and pumping stdio with `tokio::io::copy`. |
| 135 | |
| 136 | `serve_clone` and `serve_push` use cases enforce visibility and actor checks **before |
| 137 | any protocol byte flows** — that ordering is the whole point of putting them in the |
| 138 | application layer. |
| 139 | |
| 140 | #### SSH channel bridging |
| 141 | |
| 142 | The fiddly part, and worth re-reading before Milestone 3: |
| 143 | |
| 144 | - Store `Channel<Msg>` per `ChannelId` in the handler's map on `channel_open_session` |
| 145 | - On `exec_request`, take the channel, split it with `into_stream()` + |
| 146 | `tokio::io::split` |
| 147 | - Take stderr via `make_writer_ext(Some(1))` **before** `into_stream()` — that call |
| 148 | consumes the channel, so the order is not optional |
| 149 | - No `data()` or `channel_eof()` handlers needed once the streams are split |
| 150 | |
| 151 | An earlier iteration used mpsc channels, custom `ChannelReader`/`ChannelWriter`, and a |
| 152 | `spawn_blocking` thread. All of it was deleted and the result was simpler. |
| 153 | |
| 154 | ### SSH key auth and authorization |
| 155 | |
| 156 | `SshKey { id, user_id, name, fingerprint, openssh }` + port, migration |
| 157 | `007_create_ssh_keys.sql` (`fingerprint` UNIQUE). Fingerprints are SHA256 via |
| 158 | `russh::keys::ssh_key::PublicKey::fingerprint(HashAlg::Sha256)`, stored as `SHA256:…`. |
| 159 | `add_ssh_key` parses the openssh blob, dedupes on fingerprint, and re-encodes to a |
| 160 | canonical form before storing. |
| 161 | |
| 162 | `auth_none` rejects. `auth_publickey` fingerprints the offered key, looks it up, and |
| 163 | on a match stores `user_id` on the handler; `exec_request` builds the real `Actor` |
| 164 | from it. |
| 165 | |
| 166 | Authorization rules as shipped: |
| 167 | |
| 168 | |
| 169 | |
| 170 | | Clone, public repo | open | |
| 171 | | Clone, private repo | any membership in the repo's org | |
| 172 | | Push | `Role::Owner` membership in the repo's org | |
| 173 | |
| 174 | Web UI at `/{owner}/keys` — owner-only, lists fingerprints, accepts openssh via |
| 175 | textarea, revokes per-row. |
| 176 | |
| 177 | **Verified end-to-end:** clone with an unregistered key → `Permission denied` (exit |
| 178 | 128); register via web UI → clone and push both succeed; second unregistered key → |
| 179 | rejected at auth; revoke via web UI → subsequent clone rejected at auth. |
| 180 | |
| 181 | ### Security notes |
| 182 | |
| 183 | Attempt #2 ran with a **named, deliberate backdoor** between milestones: SSH accepted |
| 184 | any connection and passed a placeholder `Actor` (`UserId("ssh-anonymous")`), leaving |
| 185 | push open to anyone who could reach the port. It was recorded with an explicit |
| 186 | tightening point (`ssh.rs::exec_request`) and a closing milestone, and it did close. |
| 187 | |
| 188 | That practice is worth keeping. When this attempt opens a hole to make progress, name |
| 189 | it, name the line that closes it, and name the milestone. |
| 190 | |
| 191 | ### Never built |
| 192 | |
| 193 | Repo browsing (tree/blob/log), HTTP smart protocol, personal access tokens, flash |
| 194 | messages, issues, PRs, blogs, pages, project showcases. Milestones 5–8 in |
| 195 | [ROADMAP.md](ROADMAP.md) are all greenfield. |