Merge pull request #492 from Extra-Chill/fix/nonrepeatable-issue-list

Fix repeated GitHub issue list calls

Author: chubes4

inc/Tools/GitHubTools.php

@@ -260,8 +260,7 @@ public function handleListIssues( array $parameters, array $tool_def = array() )
      */
     public function getListIssuesDefinition(): array
     {
-        return $this->repeatableDefinition(
-            array(
+        return array(
             'class'       => __CLASS__,
             'method'      => 'handleListIssues',
             'description' => 'List issues from a GitHub repository. Returns issue numbers, titles, states, labels, assignees, comment counts, timestamps, and a generated_at timestamp. Use to review open issues, track progress, or find specific issues by label.',
@@ -287,7 +286,6 @@ public function getListIssuesDefinition(): array
             ),
             'required'   => array( 'repo' ),
             ),
-            ) 
         );
     }
 

tests/smoke-github-create-tools.php

@@ -218,6 +218,9 @@ function is_wp_error( $thing ): bool
     $assert('remove_label_from_issue is available in chat', in_array('chat', $github_tools->registered['remove_label_from_issue']['contexts'] ?? array(), true));
     $assert('remove_label_from_issue is available in pipeline', in_array('pipeline', $github_tools->registered['remove_label_from_issue']['contexts'] ?? array(), true));
 
+    $list_issues_definition = $github_tools->getListIssuesDefinition();
+    $assert('list_github_issues uses default duplicate guard', 'repeatable' !== ( $list_issues_definition['runtime']['duplicate_policy'] ?? '' ));
+
     $issue_definition = $issue_tool->getToolDefinition();
     $issue_params     = $issue_definition['parameters']['properties'] ?? array();
     $assert('create_github_issue exposes assignees from ability schema', array_key_exists('assignees', $issue_params));