Skip to content

Fix exposed private identifiers and hardcoded machine-specific paths - #4

Draft
ignasxv with Copilot wants to merge 2 commits into
mainfrom
copilot/review-exposed-secrets
Draft

Fix exposed private identifiers and hardcoded machine-specific paths#4
ignasxv with Copilot wants to merge 2 commits into
mainfrom
copilot/review-exposed-secrets

Conversation

Copilot AI commented Mar 9, 2026

Copy link
Copy Markdown

Private Notion workspace IDs and machine-specific absolute paths (embedding username ignasxv) were committed to the public repository. Two auto-generated, machine-specific files were also being tracked unnecessarily.

Changes

Secrets / private identifiers

  • claude/.claude/notion.md: Redacted private Notion database URL and data source ID → replaced with YOUR_DATABASE_ID / YOUR_DATA_SOURCE_ID placeholders

Hardcoded paths → portable equivalents

  • fish/conf.d/turso.fish: /home/ignasxv/.turso$HOME/.turso

Machine-specific files removed from tracking

  • fish/.config/fish/fish_variables: Auto-generated fish universal variables file; contained hardcoded PATH entries with absolute username paths — added to .gitignore and untracked
  • claude/.claude/plugins/known_marketplaces.json: Auto-generated by Claude tooling; stored absolute installLocation paths with username — added to .gitignore and untracked
Original prompt

This section details on the original issue you should resolve

<issue_title>review</issue_title>
<issue_description>@copilot review any exposed secretd and potential danger in the repository or any bad practices</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: ignasxv <62844231+ignasxv@users.noreply.github.com>
Copilot AI changed the title [WIP] Review exposed secrets and potential dangers Fix exposed private identifiers and hardcoded machine-specific paths Mar 9, 2026
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.

review

2 participants