jackwener/maka-agent

refactor(computer-use): extract TOCTOU target re-verification from cua-driver-backend.ts

Fermée

#1 411 ouverte le 23 juil. 2026

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)TypeScript (0 fork)github user discovery
enhancementgood first issue

Métriques du dépôt

Stars
 (1 étoile)
Métriques de merge PR
 (Métriques PR en attente)

Description

Part of #1084 (second-pass sink-file and boundary governance). The file grew to 2734 lines; cua-driver-snapshot.ts (233 lines) is the ready-made mirror for the extraction pattern. Scope re-verified against current main.

Scope

Move the target staleness/occlusion re-verification cluster from cua-driver-backend.ts into cua-driver-target-resolution.ts, injected the same way as the snapshot module:

  • validateSemanticElementVisibility (~L1073, incl. the target_occluded winner logic)
  • validateStoredWindow (~L1155)
  • boundWindowPoint (~L1268)
  • validateBoundCoordinate (~L1294, stale_frame)
  • coordinateTarget (~L1432)
  • refetchSemanticElement (~L1442)
  • the stale_frame/target_occluded error branches (~L1147/1421/1481–1508/1951–1963)

Invariants

Same as #1084: behavior-neutral; stable exports; no reverse imports; flat PR off main.

Verification

Typecheck/build + the existing __tests__/cua-driver-backend.test.ts coverage.

Guide contributeur

Direction de recherche
Examinez le modèle d'extraction de la PR #1084 (extraction du module snapshot). Localisez chaque fonction listée dans cua driver backend.ts et déplacez les vers cua driver target resolution.ts en suivant le même mécanisme d'injection. Assurez vous que le comportement est neutre et les exportations stables.
Stack technique
typescript
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75