Skip to content

Improve MRU gist labels as owner/filename with collision refs#38

Merged
kzu merged 1 commit into
mainfrom
dev/gist-mru
Jul 22, 2026
Merged

Improve MRU gist labels as owner/filename with collision refs#38
kzu merged 1 commit into
mainfrom
dev/gist-mru

Conversation

@kzu

@kzu kzu commented Jul 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Render gist MRU picker entries as GitHub-style owner/filename instead of the full host URL.
  • Persist the resolved entry filename on HistoryEntry when recording so labels remain useful after the download cache is cleaned.
  • When two or more gists share the same owner and file name, disambiguate with the familiar short-ref form owner/shortsha:file (first 7 chars of the gist id).
  • Document multi-file gist :path selection and the new MRU labeling in the main readme.

Test plan

  • Unit tests cover path suffix, cached entry, collision disambiguation, HTTPS URL form, capture on Record, and fallback when the file is unknown.
  • dotnet test --filter RunHistoryTests passes.
  • Manually: run a gist without :path, open MRU picker (dnx go), confirm owner/file label.
  • Manually: run two gists with the same entry file name, confirm owner/shortsha:file labels.

Render gist history entries like GitHub (owner/filename) instead of the
full host URL. Persist the resolved entry file on HistoryEntry so labels
survive cache cleanup. When multiple gists share the same owner and file,
disambiguate with owner/shortsha:file using the first 7 chars of the gist
id. Document multi-file gist :path selection and the new MRU labeling.
@kzu kzu added the enhancement New feature or request label Jul 22, 2026
@kzu
kzu enabled auto-merge (rebase) July 22, 2026 04:03
@kzu

kzu commented Jul 22, 2026

Copy link
Copy Markdown
Member Author

93 passed 93 passed

🧪 Details on Ubuntu 24.04.4 LTS

from retest v1.1.0 on .NET 10.0.10 with 💜 by @devlooped

@kzu
kzu merged commit 7384e5b into main Jul 22, 2026
4 checks passed
@kzu
kzu deleted the dev/gist-mru branch July 22, 2026 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant