docs(attribution): align data-visualization license summaries (stacked on #112) - #113
Open
erinepshovel-code wants to merge 3 commits into
Open
erinepshovel-code wants to merge 3 commits into
erinepshovel-code wants to merge 3 commits into
Conversation
Upstream anthropics/knowledge-work-plugins@94e1a08 ships only data/skills/data-visualization/SKILL.md. The examples/, references/, information_design_audit.py, and visual-grammar.json files were added locally in skill-lib 868de86 and are original MPL-2.0 content. Records the appended information-design extension as a local modification of the imported SKILL.md. Regenerates skill-lib_msdmd.ts with msdmd.collect because the snapshot identity binds ATTRIBUTION.md content (collection --check requires it). Flagged in The-Interdependency/stack#60.
Follow-up to #112 review: - README, CLAUDE.md, ORG_DISTRIBUTION.md, and the SKILL.md Provenance now say only SKILL.md is imported (anthropics/knowledge-work-plugins @94e1a08); the whole SKILL.md, including local additions, is Apache-2.0; the four extension files from 868de86 are MPL-2.0. - Provenance notes removal of the Cowork-only user-invocable key. - ATTRIBUTION.md lists the four local files individually, tightens the extension sentence, and records the License decision that local additions to SKILL.md are contributed under Apache-2.0. - Regenerates skill-lib_msdmd.ts for the changed hashed files.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
erinepshovel-code
changed the base branch from
repair/data-visualization-attribution
to
main
September 28, 2026 23:39
…tion-attribution-followup # Conflicts: # ATTRIBUTION.md # skill-lib_msdmd.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses the review findings on #112 without pushing to #112, so its approval stays in place.
README.md,CLAUDE.mdandORG_DISTRIBUTION.mdrows and the## Provenancesection ofdata-visualization/SKILL.mdnow matchATTRIBUTION.md:SKILL.mdis imported fromanthropics/knowledge-work-plugins@94e1a08.SKILL.md, including its local additions, is Apache-2.0.868de86are original to skill-lib under MPL-2.0.SKILL.md. This was settled by the License agent's decision, not left ashmmm: the local additions are Apache-2.0.## Workflow,## Anti-patterns,## The Interdependency information-design extensionand its subsections, and## Provenance) are contributed under Apache-2.0, so the whole file is available under Apache-2.0. I confirmed that all of the named headings exist in the file.ATTRIBUTION.mdgains the matching License-provided sentence, verbatim.user-invocablefrontmatter key was removed.ATTRIBUTION.mdlists the four local files individually instead of theexamples/andreferences/folders.ATTRIBUTION.mdis tightened with the same meaning: "data-visualization/SKILL.mdalso has a clearly marked information-design extension appended locally (skill-lib868de86)."skill-lib_msdmd.ts. Regenerated withpython -m msdmd.collect … --strictbecause hashed files changed. The diff is the new snapshot hash plus new content hashes and sizes forATTRIBUTION.md,CLAUDE.md,ORG_DISTRIBUTION.md,README.mdanddata-visualization/SKILL.md.No licence headers or other files changed.
Verification
--checkandtsc --noEmit --strict: passpython -m unittest discover -s tests: 377 passed (Python 3.13, pinnedmsdmd/requirements.txt, Node 24.15.0)--warnings-fail, codex adapters--check,llms.build --check, RATIOS--strict, gonol authority, RepoLOTO audit and checks, no tracked bytecode,git diff --check: all passhmmm
ATTRIBUTION.md, such as stack (see stack#60), keep the old wording until they next sync. That sync is intentionally deferred until docs(attribution): narrow data-visualization import to SKILL.md #112 and this PR land.