Skip to content

chore(deprivatize): scrub private references for public release#19

Open
m-szymanska wants to merge 3 commits into
mainfrom
chore/deprivatize
Open

chore(deprivatize): scrub private references for public release#19
m-szymanska wants to merge 3 commits into
mainfrom
chore/deprivatize

Conversation

@m-szymanska

Copy link
Copy Markdown
Collaborator

Deprivatize pass for public-release hygiene: scrub private references
(personal names, operator paths, private org/infra fingerprints) per the
cross-repo cleanup.

Verified by a read-only leak audit over tracked files at HEAD: no residual
personal names, operator home paths, host names, tailnet IPs, private
endpoints, or secrets. Brand/product identifiers (vetcoders, loctree, etc.)
intentionally retained.

🤖 Generated with Claude Code

m-szymanska and others added 2 commits June 28, 2026 20:22
Remove the .loctree/context-atlas directory (loctree-generated internal
context atlas embedding real home paths, monorepo layout, timestamps) and
AGENTS.md (operator-internal loctree/aicx/vibecrafted doctrine and Vista
codename). Add .loctree/ to .gitignore so the artifact never re-enters.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Collapse personal/agent attribution to vetcoders (LICENSE copyright,
README/README_PL 'Developed by', glama maintainers, issue-template assignee,
CONTRIBUTING clone URL). Replace personal domain/email (div0.space,
void@div0.space) with vetcoders.io / hello@vetcoders.io in package.json,
package-lock.json and SECURITY.md. Normalize camelCase brand (VetCoders,
VibeCrafted) to lowercase in URLs and Capital-first standalone across docs
and metadata. Upstream MIT attribution (stefanskiasan) and external
contributor credits are preserved.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request cleans up the repository by removing the precomputed .loctree context atlas files and adding .loctree/ to .gitignore. It also standardizes the repository owner/organization name to lowercase vetcoders across all documentation, configuration, and package files, updating contact emails and funding URLs accordingly. As there are no review comments provided, I have no additional feedback to offer.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 555a33caf3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread LICENSE
MIT License

Copyright (c) 2025 Szowesgad
Copyright (c) 2025 Vetcoders

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve original MIT copyright notice

For code that was already distributed under the previous MIT notice, replacing the existing copyright holder with Vetcoders drops the notice that the license text below requires downstream copies to retain. Unless there was an explicit copyright transfer for all prior contributions, this should preserve the original notice and add any new organization notice separately so redistributed packages keep valid attribution.

Useful? React with 👍 / 👎.

The self-hosted runner's system Python is now externally-managed (PEP 668), so
`pip install semgrep` fails with externally-managed-environment before semgrep
even runs. Install into a per-run venv and put it on PATH instead — isolated,
no system site-packages pollution. Pre-existing infra breakage, unrelated to
the de-privatization scrub.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f52bcb3528

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +42 to +43
"${{ steps.python.outputs.python }}" -m venv "$VENV" \
|| "${{ steps.python.outputs.python }}" -m ensurepip --upgrade && "${{ steps.python.outputs.python }}" -m venv "$VENV"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Restore a fallback when venv is unavailable

On self-hosted Linux runners that have pip installed but not the venv module/package, this now fails before Semgrep is installed even though the previous script would proceed with pip install. I checked the Python helpers here: venv creates the environment and bootstraps pip, while ensurepip only bootstraps pip, so retrying ensurepip cannot add a missing venv capability; this affects common Debian/Ubuntu-style Python installs without python3-venv.

Useful? React with 👍 / 👎.

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