Skip to content

Avoid collecting unused Git metadata in skill analytics - #49076

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/f878936937d0ff8c6313bb8168448e0b25ba1a39
Sep 28, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/f878936937d0ff8c6313bb8168448e0b25ba1a39

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Avoid collecting unused Git metadata in skill analytics

Why

Skill invocation analytics only needs the repository's origin URL, but
collect_git_info also checks the repository and reads its commit and branch.

What changed

Extract and export get_git_origin_url and use it directly in skill analytics
to skip the extra Git commands. Reuse the helper in collect_git_info,
preserving URL sanitization and command timeouts.

## Why

Skill invocation analytics only needs the repository's origin URL, but
`collect_git_info` also checks the repository and reads its commit and branch.

## What changed

Extract and export `get_git_origin_url` and use it directly in skill analytics
to skip the extra Git commands. Reuse the helper in `collect_git_info`,
preserving URL sanitization and command timeouts.

GitOrigin-RevId: f878936937d0ff8c6313bb8168448e0b25ba1a39
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/f878936937d0ff8c6313bb8168448e0b25ba1a39 branch from 74ec0c5 to 011f803 Compare September 28, 2026 20:45
@copyberry
copyberry Bot merged commit 011f803 into main Sep 28, 2026
18 of 24 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/f878936937d0ff8c6313bb8168448e0b25ba1a39 branch September 28, 2026 20:46
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 28, 2026

This branch was previously deployed

1 inactive deployment
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants