Fix hook metadata emission when class deps are unresolved

[twisti-dev]

Motivation

• Avoid emitting incomplete or duplicate hook metadata when @DependsOnClass references are unresolved so generated surf-hooks.json does not contain entries missing class dependencies.

Description

• In surf-api-gradle-plugin/surf-api-processor/src/main/kotlin/.../HookSymbolProcessor.kt track unresolved class dependencies with a hasUnresolvedClassDependency flag and short-circuit mapNotNull to return@mapNotNull null when any DependsOnClass reference is unresolved or its closestClassDeclaration() is missing, preventing partial PluginHookMeta.Hook entries.

Testing

• No automated tests were run for this change.

---

Codex Task

[github-actions]

⚠️ API/ABI changes detected!

This PR contains changes that modified the public API. To update the reference ABI dumps:

./gradlew updateLegacyAbi
git add **/api/**
git commit -m "Update ABI reference"
git push

After updating, the CI will pass. Make sure the changes are backward compatible.

[Copilot]

Pull request overview

This PR fixes an issue where hook metadata could be incomplete when @DependsOnClass annotation references are unresolved during symbol processing. The change ensures that hooks with unresolved class dependencies are properly deferred for later processing rather than being partially emitted to the generated surf-hooks.json file.

Changes:

• Added a hasUnresolvedClassDependency flag to track when any @DependsOnClass annotation has an unresolved reference
• Added early return logic after processing all class dependencies to skip hooks with unresolved dependencies
• Ensured unresolved hooks are added to the deferred list for reprocessing in subsequent rounds

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

If a hook class has multiple @DependsOnClass annotations with unresolved dependencies, hookClass will be added to the deferred list multiple times (lines 57 and 64). This could lead to duplicate entries in the deferred list and potentially reprocessing the same hook multiple times in subsequent rounds.

Consider adding the hook to deferred only once, either by checking if it's already in the list before adding, or by deferring the addition until after all annotations are processed (e.g., moving it to lines 71-73 where the flag is checked).