chore: remove stray teamai init test artifacts from repo root - #284
Merged
Conversation
Commit 2117de1 accidentally committed the output of running `teamai init` against this repo itself: a member profile, a team teamai.yaml, and empty skills/rules/env/docs/members scaffolding dirs. None of it is product code or a test fixture (all code reads teamai.yaml from a runtime/tmp repoPath, never the repo root). Remove the 7 stray files. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
jeff-r2026
added a commit
that referenced
this pull request
Aug 6, 2026
Resolve merge conflict in src/init.ts (keep both resolveDotTarget and resolveInheritUserScope). The stray dogfooding artifacts (teamai.yaml, members/jeff-r2026.yaml, .gitkeep placeholders) are removed by main's #284. Fixes from PR #277 review: - pull.ts: guard auto-bootstrap with provider.isAuthenticated() so the session-start hook never triggers an interactive `gh auth login --web` that would hang until the hook timeout. - pull.ts: normalize $HOME collision guard via resolveRealPath (matches init's resolveInitScope) instead of raw string compare. - pull.ts: drop redundant hook reconcile inside bootstrap; the normal pull flow reconciles once via reconcileHooksAllScopes. - git.ts: normalizeRemoteUrl now handles `ssh://git@host[:port]/path` remotes; export shared resolveRealPath (init.ts reuses it). - hooks.ts: removeAll still cleans up shell-dependent (codebuddy/workbuddy) hooks when /bin/sh is absent, instead of skipping them. - docs: note that auto-bootstrap skips interactive login when unauthenticated. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
What
Remove 7 stray files that were accidentally committed to the repo root by
2117de1("[teamai] Register member: jeff-r2026").Why
2117de1committed the output of runningteamai initagainst this repo itself — dogfooding artifacts, not product code:teamai.yaml— a team config stub (team: my-team, points repo at this project)members/jeff-r2026.yaml— a test member profiledocs/.gitkeep,env/.gitkeep,members/.gitkeep,rules/.gitkeep,skills/.gitkeep— empty scaffolding dirsteamai initcreates for a team-resource repoNone of this is product source or a test fixture. Every code/test reference to
teamai.yamlresolves it from a runtime/tmprepoPath(src/config.ts,src/source.ts,src/init.ts, tests write to tmpdirs) — never the repo root. So these files are inert clutter that also misleads anyone browsing the repo into thinking it ships a team config.How
New deletion commit — history is not rewritten (this branch is on the public upstream + mirrored to TGit, so a rebase/reset is off the table).
Test plan
npm run build— successnpx tsc --noEmit— cleannpx vitest run— 147 files / 1946 tests passed (no fixture depended on the removed files)🤖 Generated with Claude Code