Skip to content

Credit the people whose public work this repo borrows - #63

Merged
nonreagent merged 2 commits into
mainfrom
credit-find-inspiration-sources
Sep 13, 2026
Merged

nonreagent merged 2 commits into
mainfrom
credit-find-inspiration-sources

Conversation

@nonreagent

@nonreagent nonreagent commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Follows up #31

Problem

#31's definition of done promised README credit once adopted find-inspiration ideas landed. Ten have landed from five repos (#6, #7, #22, #23, #24, #30, #34, #38, #40, #47), and the README credited nobody. Three more authors were credited only in a source: frontmatter line or a submodule URL.

Motivation

The rule lived only in #31. Adopted items land in later sessions that work from the issue and never load the skill, so nothing reminded them. Borrowing from people's public work in public obliges saying where it came from, and a README is the credit a stranger can see.

Proposed Solution

Two commits:

  1. README Credits section. Eight authors, one line each (avatar, name, repo link), so it reads the same on mobile; GitHub strips CSS, so a table can't reflow. They are the five find-inspiration sources plus Alex Hillman (teach), Matt Pocock (vendored skills) and chriskempson (tomorrow-theme). Credit is by author, not by item, so the section doesn't drift as borrowed ideas change shape after they land.
  2. Credit reminder in the issue body. issue_body() appends an "On landing" section telling whoever closes the issue to add the source's author if they aren't listed yet. Phase 7 of SKILL.md says why the reminder lives in the issue rather than the skill.

Checked: each commit passes make test, the skill checks, and editorconfig on its own (editorconfig-checker via uvx, since it isn't installed on this VM). The list renders through GitHub's markdown API with all eight avatars, and every avatar URL returns 200.

Feedback

  • The priors run is credited to obra. decisions.jsonl logs it as priors/known-candidates, with no repo. obra/dotfiles has the manifest, per-file test/test_*.sh suite, macos-defaults/desired.tsv, mise and zsh configs behind five of its six ids, all first committed 2026-06-04 to 06-07, before the 2026-07-04 run. I left the log and run file unrelabelled; say if you'd rather correct them.
  • Vendored authors are included. Drop them if Credits should cover only find-inspiration sources.
  • chriskempson shows the login, since the account has no display name.
  • Open issues don't get the new section. The script never mutates filed issues, so the 18 open ones rely on the Phase 7 note.

@nonreagent
nonreagent force-pushed the credit-find-inspiration-sources branch from 2625306 to 676937b Compare September 11, 2026 22:41
@nonreagent

Copy link
Copy Markdown
Collaborator Author

Self-review before handoff. An independent pass checked every credited item against decisions.jsonl, each issue's "Source:" line, its close reason, and the change that landed it.

Fixed, amended into the owning commits (now 9445aa2 and 676937b):

Held up: all eight issues closed as completed, and each maps to a change on main. The source attributions and all four display names match. No landed borrowed item is missing (#8's PR #57 is still open; #6 and #7 come from the priors run). The table renders through GitHub's markdown API. Each commit passes make test, the skill checks, and editorconfig on its own.

@nonreagent
nonreagent force-pushed the credit-find-inspiration-sources branch from 676937b to 9e18b6a Compare September 11, 2026 22:52
@nonreagent nonreagent changed the title Credit the dotfiles find-inspiration borrowed from Credit the people whose public work this repo borrows Sep 11, 2026
@nonreagent

Copy link
Copy Markdown
Collaborator Author

Reworked to credit authors rather than items, now a73f753 and 9e18b6a. This supersedes the #22 row fix above: the per-item lists are gone.

Also added obra. The 2026-07-04 priors run, logged without a repo, traces to obra/dotfiles by file and commit date, and /par already carried obra's source: line. That brings the landed count to ten items from five repos.

@nonreagent
nonreagent force-pushed the credit-find-inspiration-sources branch from 9e18b6a to a9434e2 Compare September 11, 2026 23:00
@nonrational
nonrational force-pushed the credit-find-inspiration-sources branch 4 times, most recently from 5b89b4c to caf8e6b Compare September 13, 2026 15:47
nonreagent and others added 2 commits September 13, 2026 08:51
Issue #31 promised README credit once adopted find-inspiration ideas
landed. Ten have, from five repos, and the README credited nobody.
Three more authors were credited only in frontmatter or a submodule URL.
The credit rule lived only in #31, so the sessions that landed adopted
items never saw it. Those sessions read the issue, not the skill, so
the reminder now travels in the issue body, and Phase 7 says why.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@nonrational
nonrational force-pushed the credit-find-inspiration-sources branch from caf8e6b to 5ca421b Compare September 13, 2026 15:52
@nonreagent
nonreagent merged commit 17170d7 into main Sep 13, 2026
2 checks passed
@nonreagent
nonreagent deleted the credit-find-inspiration-sources branch September 13, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants