Skip to content

chore: gitignore the .local scratch directory - #2961

Open
DevMomo wants to merge 1 commit into
mainfrom
chore/gitignore-local
Open

chore: gitignore the .local scratch directory#2961
DevMomo wants to merge 1 commit into
mainfrom
chore/gitignore-local

Conversation

@DevMomo

@DevMomo DevMomo commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Problem

.local/ is a documented convention in this repo — the uipath-troubleshoot skill writes investigation artifacts there, including raw/, which holds verbatim CLI responses from a live tenant:

tests/tasks/uipath-troubleshoot/CLAUDE.md:238
  The skill writes investigation artifacts to `.local/investigations/` — NOT `.investigations/`.

It isn't ignored, so anyone who runs that skill is one git add . from committing tenant data:

$ git check-ignore .local/     # no match

Fix

Ignore .local/, with a comment naming why. Nothing tracked is affected — .local/ has never been committed.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Claude finished @DevMomo's task in 49s —— View job


Summary

Adds .local/ to .gitignore to prevent scratch projects and CLI probe artifacts — which contain real tenant identifiers (connection IDs, folder keys, account names) — from being accidentally staged via git add -A.

Change-by-Change Review

1. .gitignore — Lines 92–94

Severity: OK

The entry is correctly placed between the existing .worktrees/ block and the Node section, maintaining the file's logical grouping. The two-line comment clearly explains why the directory is ignored (real tenant identifiers), which is appropriate context for a security-motivated ignore rule. The trailing-slash pattern (.local/) correctly targets only the root-level directory.

What's Missing

Nothing identified.

Area Ratings

Area Status Notes
Frontmatter N/A No skill files changed
E2E Tests N/A No skill files changed
Skill Body N/A No skill files changed
References & Assets N/A No skill files changed
Repo Hygiene OK Prevents accidental commit of sensitive local artifacts

Issues for Manual Review

None found.

Conclusion

Clean, well-scoped change. The .local/ ignore entry prevents accidental exposure of real tenant identifiers. No issues — LGTM.


  • Gather context and read review criteria
  • Read the full diff
  • Read the full .gitignore file
  • Check for secrets/sensitive data in the PR
  • Post review findings

@DevMomo
DevMomo force-pushed the chore/gitignore-local branch from d6cd58d to d9f9f62 Compare September 7, 2026 08:33
.local/ holds local scratch projects and CLI probe artifacts, including real
connection IDs, folder keys, and account names. It was untracked but not
ignored, so any `git add -A` swept it into a commit.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015DDDLGA54192tVTXiCgxEL
@DevMomo
DevMomo force-pushed the chore/gitignore-local branch from d9f9f62 to f5edf2a Compare September 7, 2026 20:05
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