~ knotless Search/
↑↓ move↵ open⇧↵ all results

chadtmiller.com/knotless

git that lives in your Atmosphere account

knotless.dev
1 collaborator

knotless 0.13.0 Latest 2 packages, 1 image

1
10

Space the Private tag off a repository's name in lists

Merged
Merged chadtmiller.com asked to merge into main from chadtmiller.com/knotless:web/private-tag-space

The Private tag on a repository card, and in an account's list of repositories, sat right against the name. It now has a space before it, as it does in the repository's header.

Claude Opus 5.5 · high

What changes

RepoCards, ProfileRepos and RepoList draw a space between a repository's name and its Private tag.

Why

Svelte drops whitespace at the start of an {#if} block, so the line break before <PrivateTag /> in >{#if r.private} never became a space. Each now writes {' '} before the tag, as StepSheet.svelte and the pull page already do for theirs.

How it was checked

just look profile and just look profile-repos with --set 'getActorRepos.repos.0.private=true', desktop light and phone dark: the tag stands a space off the name, as in the header. The recorded world has no private repository, so no shot should move.

Shortcomings

RepoList (search results and stars) was not looked at; its markup was the same as the other two.

profile-phone-dark.png

Approved

The Svelte change is correct. Svelte trims whitespace at the start of an {#if} block, so {' '} before <PrivateTag /> is the right fix in all three components. I read every changed file.

The pull also carries a second, unrelated commit, "An agent watching an owner finds their private repositories shared with it". It is not in main, and neither the title nor the body mentions it. It covers knotless-agent --owner finding invited private repositories, repo share leaving an invite, the changeset, the docs, a test, and removing group-host from deploy.yml.

I read that commit and found nothing wrong. The invite paging loop in crates/knotless-agent/src/lib.rs matches the triager's. The appview always returns a cursor, so it ends on the empty page. It would be cleaner to land that commit in its own pull, or to say in this one's body that it is stacked on it.

RepoList was not checked visually, but its markup is identical to the other two.

Merged as c3f30b7