Skip to content

Remove cuyz/valinor from composer.json#3

Merged
varonix0 merged 1 commit intoInfisical:mainfrom
adeildo-jr:main
Apr 20, 2026
Merged

Remove cuyz/valinor from composer.json#3
varonix0 merged 1 commit intoInfisical:mainfrom
adeildo-jr:main

Conversation

@adeildo-jr
Copy link
Copy Markdown
Contributor

Removed cuyz/valinor from the required dependencies.

The dependency is not being used and is creating conflicts with others packages, like Firebase.

Removed cuyz/valinor from the required dependencies.
@adeildo-jr
Copy link
Copy Markdown
Contributor Author

@varonix0 may you please take a look into this? Thanks.

@adeildo-jr adeildo-jr mentioned this pull request Mar 31, 2026
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Mar 31, 2026

Greptile Summary

This PR removes the cuyz/valinor package from the require section of composer.json. A full codebase scan confirms that no source file in src/ imports or references any class from CuyZ\Valinor, so the dependency was genuinely unused and its removal carries no functional risk. This is a clean dependency hygiene fix that resolves reported conflicts with packages such as Firebase.

  • Dependency removed: cuyz/valinor ^1.0 from composer.json require
  • Usage check: No references to CuyZ\\Valinor, MapperBuilder, TreeMapper, or any other Valinor symbol found across the entire repository
  • Impact: None on existing functionality; downstream consumers will no longer need to resolve this transitive dependency tree

Confidence Score: 5/5

Safe to merge — removes a confirmed-unused dependency with no functional impact

The change is a single-line removal of a dependency that is verifiably unused across the entire codebase. No source files import or reference cuyz/valinor, no breaking changes are introduced, and no other files were modified. All remaining findings are non-existent — there are no P0, P1, or P2 issues.

No files require special attention

Important Files Changed

Filename Overview
composer.json Removes unused cuyz/valinor dependency; no references to this package exist anywhere in the source tree, confirming the removal is safe

Reviews (1): Last reviewed commit: "Remove cuyz/valinor from composer.json" | Re-trigger Greptile

@varonix0 varonix0 merged commit d69ab62 into Infisical:main Apr 20, 2026
5 checks passed
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.

2 participants