Skip to content

chore(repo): make docs/ team-private + untrack internal artifacts#25

Merged
Hyeongseob91 merged 1 commit into
feat/wigtn-ppt-skillfrom
chore/docs-team-private
Jun 27, 2026
Merged

chore(repo): make docs/ team-private + untrack internal artifacts#25
Hyeongseob91 merged 1 commit into
feat/wigtn-ppt-skillfrom
chore/docs-team-private

Conversation

@Hyeongseob91

Copy link
Copy Markdown
Contributor

Summary

Makes docs/ team-private so brand assets, PPT templates/data, and internal planning notes never ship in this public Apache-2.0 plugin — and documents the rule so agents/contributors don't re-introduce them.

Stacked on #24 (feat/wigtn-ppt-skill). Base will retarget to main once #24 merges. Review/merge #24 first.

Changes

  • .gitignore: ignore docs/ (brand logos, ppt templates & data, internal PRD/PLAN/migration notes)
  • Untrack already-committed docs/ files (a teammate added them in chore!: rename to wigtn-plugins (plugin id + repo + title) #21 / earlier) — git rm --cached, files kept on disk:
    • docs/opus-4-8-migration.ko.md
    • docs/prd/PRD_auto-pr-reviewer.md
    • docs/todo_plan/PLAN_auto-pr-reviewer.md
  • CLAUDE.md: new "Repository Hygiene — team-private artifacts (NEVER commit)" section as the backstop (ignore rules don't catch already-tracked files), with the untrack procedure

Why

Ignore rules alone don't help once a file is tracked, so the documented rule + the one-time untrack together prevent private material from being published.

Quality Gate

  • Score: 95/100 (PASS)
  • Verified no source/manifest changes leak in (scope = .gitignore + CLAUDE.md + docs deletions only)
  • Files remain on disk for team use; brand PNGs stay gitignored in both docs/images/ and the skill's assets/logo/

Test Plan

  • After merge, git ls-files docs/ returns empty
  • git status shows docs/ as ignored (no longer tracked)
  • Local docs/ files still present for team members

🤖 Generated with Claude Code

https://claude.ai/code/session_01VDvYizKVU4qf57HJjzN6iY

docs/ holds team-private material (WIGTN brand logos, PPT templates & data,
internal PRD/PLAN/migration notes) that should not ship in this public
Apache-2.0 plugin. A teammate had committed some of it by accident.

- gitignore docs/ so new private material never gets staged
- untrack already-committed docs (PRD/PLAN/migration); files kept on disk
- document a "Repository Hygiene" rule in CLAUDE.md as the backstop for
  ignore rules (which don't catch already-tracked files)

Quality Score: 95/100
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VDvYizKVU4qf57HJjzN6iY
@Hyeongseob91 Hyeongseob91 merged commit 8f58649 into feat/wigtn-ppt-skill Jun 27, 2026
1 check passed
Hyeongseob91 added a commit that referenced this pull request Jun 27, 2026
* chore(release): bump to v0.1.9 and document wigtn-ppt in READMEs

- bump version 0.1.8 -> 0.1.9 across all manifests, README badges, CLAUDE.md
- add wigtn-ppt row to the Skills table in README (en/ko/cn); bump skill
  count to 6 in each
- restore the missing handdrawn-diagram row in README.cn (was 4, now 6)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VDvYizKVU4qf57HJjzN6iY

* chore(repo): make docs/ team-private and untrack internal artifacts

Re-land the team-private hygiene change on main: PR #25 was merged into the
feature branch, not main, so its content never reached the default branch.

- gitignore docs/ (brand logos, ppt templates & data, internal notes)
- untrack already-committed docs (PRD/PLAN/migration); files kept on disk
- document the "Repository Hygiene" rule in CLAUDE.md as the backstop

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VDvYizKVU4qf57HJjzN6iY

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant