Skip to content

Remove private mobile-geolocation repository entry#4

Merged
simonhamp merged 1 commit into
mainfrom
remove-private-plugin-repo
Jul 13, 2026
Merged

Remove private mobile-geolocation repository entry#4
simonhamp merged 1 commit into
mainfrom
remove-private-plugin-repo

Conversation

@simonhamp

Copy link
Copy Markdown
Member

Why

The geolocation VCS repository in composer.json points at nativephp/mobile-geolocation, which is a private GitHub repo. Composer contacts every configured repository during dependency resolution, so anyone cloning this project and running composer update gets prompted for GitHub credentials — even though nothing in require uses the package.

What

  • Removed the geolocation repository entry (the package is served publicly by plugins.nativephp.com if it's ever needed again)
  • Regenerated the composer.lock content-hash (composer update --lock, zero package changes)

What stays and why

  • mobile (github.com/nativephp/mobile-air): public repo, and required — this app pins nativephp/mobile to dev-element, which plugins.nativephp.com does not serve (tagged releases only)
  • ui (github.com/nativephp/native-ui): public repo
  • plugins (plugins.nativephp.com): the satis repo itself

🤖 Generated with Claude Code

The geolocation VCS repository points at a private GitHub repo, so
composer prompts everyone for credentials during resolution even though
nothing in require depends on it. The package is served publicly by
plugins.nativephp.com anyway. The remaining GitHub entries (mobile-air,
native-ui) are public and mobile-air is required to resolve
nativephp/mobile dev-element, which the plugins repo does not serve.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@simonhamp simonhamp merged commit 03c80ed into main Jul 13, 2026
3 checks passed
@simonhamp simonhamp deleted the remove-private-plugin-repo branch July 13, 2026 22:31
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