Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
type: action
definition:
title: Get campaign
description: Get a campaign from Gophish.
display_group: Gophish
doc_url: https://docs.getgophish.com/api-documentation/campaigns#get-campaigns
Copy link

@cubic-dev-ai cubic-dev-ai bot Sep 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doc_url points to the list endpoint section ("get-campaigns") instead of the single campaign action. Link to the campaigns page root or the correct anchor for single campaign.

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/get_campaign.yml at line 6:

<comment>doc_url points to the list endpoint section (&quot;get-campaigns&quot;) instead of the single campaign action. Link to the campaigns page root or the correct anchor for single campaign.</comment>

<file context>
@@ -0,0 +1,29 @@
+  title: Get campaign
+  description: Get a campaign from Gophish.
+  display_group: Gophish
+  doc_url: https://docs.getgophish.com/api-documentation/campaigns#get-campaigns
+  namespace: tools.gophish
+  name: get_campaign
</file context>

[internal] Confidence score: 8/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

namespace: tools.gophish
name: get_campaign
secrets:
- name: gophish
keys: ["GOPHISH_API_KEY"]
expects:
base_url:
type: str
description: The base URL of the Gophish instance.
required: true
campaign_id:
type: int
description: The ID of the campaign to get.
required: true
steps:
- ref: get_campaign
action: core.http_request
args:
url: ${{ inputs.base_url }}/api/campaigns/${{ inputs.campaign_id }}
method: GET
headers:
Authorization: Bearer ${{ SECRETS.gophish.GOPHISH_API_KEY }}
returns: ${{ steps.get_campaign.result.data }}
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
type: action
definition:
title: Get campaign results
description: Get a campaign's results from Gophish.
display_group: Gophish
doc_url: https://docs.getgophish.com/api-documentation/campaigns#get-campaigns
Copy link

@cubic-dev-ai cubic-dev-ai bot Sep 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doc_url anchor references listing campaigns instead of campaign results; link likely misleading.

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/get_campaign_results.yml at line 6:

<comment>doc_url anchor references listing campaigns instead of campaign results; link likely misleading.</comment>

<file context>
@@ -0,0 +1,29 @@
+  title: Get campaign results
+  description: Get a campaign&#39;s results from Gophish.
+  display_group: Gophish
+  doc_url: https://docs.getgophish.com/api-documentation/campaigns#get-campaigns
+  namespace: tools.gophish
+  name: get_campaign_results
</file context>

[internal] Confidence score: 8/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

Copy link

@cubic-staging cubic-staging bot Sep 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doc_url points to the "get-campaigns" (list campaigns) section, but this action retrieves campaign results; link is likely incorrect.

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/get_campaign_results.yml at line 6:

<comment>doc_url points to the &quot;get-campaigns&quot; (list campaigns) section, but this action retrieves campaign results; link is likely incorrect.</comment>

<file context>
@@ -0,0 +1,29 @@
+  title: Get campaign results
+  description: Get a campaign&#39;s results from Gophish.
+  display_group: Gophish
+  doc_url: https://docs.getgophish.com/api-documentation/campaigns#get-campaigns
+  namespace: tools.gophish
+  name: get_campaign_results
</file context>

[internal] Confidence score: 8/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

namespace: tools.gophish
name: get_campaign_results
secrets:
- name: gophish
keys: ["GOPHISH_API_KEY"]
expects:
base_url:
type: str
description: The base URL of the Gophish instance.
required: true
campaign_id:
type: int
description: The ID of the campaign to get results for.
required: true
steps:
- ref: get_campaign_results
action: core.http_request
args:
url: ${{ inputs.base_url }}/api/campaigns/${{ inputs.campaign_id }}/results
method: GET
headers:
Authorization: Bearer ${{ SECRETS.gophish.GOPHISH_API_KEY }}
returns: ${{ steps.get_campaign_results.result.data }}
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
type: action
definition:
title: List campaigns
description: List campaigns from Gophish.
display_group: Gophish
doc_url: https://docs.getgophish.com/api-documentation/campaigns#get-campaigns
namespace: tools.gophish
name: list_campaigns
secrets:
- name: gophish
keys: ["GOPHISH_API_KEY"]
expects:
base_url:
type: str
description: The base URL of the Gophish instance.
required: true
steps:
- ref: list_campaigns
action: core.http_request
args:
url: ${{ inputs.base_url }}/api/campaigns/
Copy link

@cubic-dev-ai cubic-dev-ai bot Sep 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove trailing slash from the endpoint to align with other Gophish commands and avoid potential redirects.

    DEV MODE: This violation would have been filtered out by GPT-5.

Reasoning:
GPT-5: Low-impact style/consistency issue; trailing slash unlikely to cause functional problems or meaningful redirects. Not worth reporting per guidelines.

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/list_campaigns.yml at line 21:

<comment>Remove trailing slash from the endpoint to align with other Gophish commands and avoid potential redirects.

        DEV MODE: This violation would have been filtered out by GPT-5.
Reasoning:
• **GPT-5**: Low-impact style/consistency issue; trailing slash unlikely to cause functional problems or meaningful redirects. Not worth reporting per guidelines.</comment>

<file context>
@@ -0,0 +1,38 @@
+    - ref: list_campaigns
+      action: core.http_request
+      args:
+        url: ${{ inputs.base_url }}/api/campaigns/
+        method: GET
+        headers:
</file context>

[internal] Confidence score: 8/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

method: GET
headers:
Authorization: Bearer ${{ SECRETS.gophish.GOPHISH_API_KEY }}

# This is needed because the API will return results and timelines for all campaigns. This can exeed the max size of workflow data steps very easily. For example a campaign with 1,000 recipients will return 11,000 lines of just results, in addition to the at least 1000 timeline items which are each another 8 lines of data. Very quickly this can exeed the max size of workflow data steps.
Copy link

@cubic-dev-ai cubic-dev-ai bot Sep 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix typo in comment: replace "exeed" with "exceed" (twice) for clarity.

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/list_campaigns.yml at line 26:

<comment>Fix typo in comment: replace &quot;exeed&quot; with &quot;exceed&quot; (twice) for clarity.</comment>

<file context>
@@ -0,0 +1,38 @@
+        headers:
+          Authorization: Bearer ${{ SECRETS.gophish.GOPHISH_API_KEY }}
+
+    # This is needed because the API will return results and timelines for all campaigns. This can exeed the max size of workflow data steps very easily. For example a campaign with 1,000 recipients will return 11,000 lines of just results, in addition to the at least 1000 timeline items which are each another 8 lines of data. Very quickly this can exeed the max size of workflow data steps.
+    - ref: parse_campaigns_list
+      action: core.script.run_python
</file context>

[internal] Confidence score: 10/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

Copy link

@cubic-staging cubic-staging bot Sep 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in comment: "exeed" should be "exceed" (appears twice on this line).

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/list_campaigns.yml at line 26:

<comment>Typo in comment: &quot;exeed&quot; should be &quot;exceed&quot; (appears twice on this line).</comment>

<file context>
@@ -0,0 +1,38 @@
+        headers:
+          Authorization: Bearer ${{ SECRETS.gophish.GOPHISH_API_KEY }}
+
+    # This is needed because the API will return results and timelines for all campaigns. This can exeed the max size of workflow data steps very easily. For example a campaign with 1,000 recipients will return 11,000 lines of just results, in addition to the at least 1000 timeline items which are each another 8 lines of data. Very quickly this can exeed the max size of workflow data steps.
+    - ref: parse_campaigns_list
+      action: core.script.run_python
</file context>

[internal] Confidence score: 10/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

- ref: parse_campaigns_list
action: core.script.run_python
args:
inputs:
data: ${{ steps.list_campaigns.result.data }}
script: |
def main(data):
return [
{"id": campaign["id"], "name": campaign["name"]}
Copy link

@cubic-staging cubic-staging bot Sep 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing whitespace at end of line. Remove to adhere to style and avoid noisy diffs.

DEV MODE: This violation would have been filtered out by screening filters. Failing filters: functionalImpact.

    DEV MODE: This violation would have been filtered out by GPT-5.

Reasoning:
GPT-5: Minor formatting-only issue; negligible impact. Filter out.

Prompt for AI agents
Address the following comment on packages/tracecat-registry/tracecat_registry/templates/tools/gophish/list_campaigns.yml at line 35:

<comment>Trailing whitespace at end of line. Remove to adhere to style and avoid noisy diffs.

*DEV MODE: This violation would have been filtered out by screening filters. Failing filters: functionalImpact.*

        DEV MODE: This violation would have been filtered out by GPT-5.
Reasoning:
• **GPT-5**: Minor formatting-only issue; negligible impact. Filter out.</comment>

<file context>
@@ -0,0 +1,38 @@
+        script: |
+          def main(data):
+            return [
+                {&quot;id&quot;: campaign[&quot;id&quot;], &quot;name&quot;: campaign[&quot;name&quot;]} 
+                for campaign in data
+            ]
</file context>

[internal] Confidence score: 9/10

[internal] Posted by: General AI Review Agent

Fix with Cubic

for campaign in data
]
returns: ${{ steps.parse_campaigns_list.result }}