The Remote struct's `name` field was being serialized to TOML even though it was already present as the map key, causing duplication like: [repos.remotes.origin] name = "origin" # redundant url = "..." Now the TOML is cleaner: [repos.remotes.origin] url = "..." Changes: - Add `#[serde(skip)]` to `Remote.name` field to exclude from serialization - Populate `name` from map keys during deserialization in `parse()` - Update all test TOML strings to use the new format The in-memory `Remote` struct still has the `name` field for convenience when iterating over remotes without map context. Fixes #248 Prompts: - "i've just noticed there is duplication in the toml, do we need the name \"origin\" repeated in the key and the values? [The toml repeats the origin name · Issue #248 · timabell/gitopolis](#248)" - "well, it seems like the struct should have \"name\" otherwise that's a bit odd, but it shouldn't be serialized out to the toml" 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>

Uh oh!

The toml repeats the origin name #248

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions