Skip to content

Support listing and deleting recorded agent actions - #6571

Open
ayoubdiourin7 wants to merge 14 commits into
mozilla:masterfrom
ayoubdiourin7:feature/manage-agent-actions
Open

Support listing and deleting recorded agent actions#6571
ayoubdiourin7 wants to merge 14 commits into
mozilla:masterfrom
ayoubdiourin7:feature/manage-agent-actions

Conversation

@ayoubdiourin7

Copy link
Copy Markdown
Contributor

Summary

  • Add tools for agents to list and remove actions recorded during the current run.
  • Assign stable action IDs so recorded actions can be referenced safely.
  • Enable the new recorded_actions tools for the bug-fix and frontend-triage agents.
  • Update prompts to explain when agents should review or retract recorded actions.
  • Add tests covering action listing, removal, unknown IDs, attachments, and final runtime summaries.

Fixes #6513

@ayoubdiourin7
ayoubdiourin7 requested review from a team as code owners August 11, 2026 16:09
Comment thread agents/frontend-triage/hackbot_agents/frontend_triage/prompts/system.md Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorder.py
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorder.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorder.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/bugzilla.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/phabricator.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorded_actions.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorded_actions.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorded_actions.py Outdated

@suhaibmujahid suhaibmujahid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think you are still working on this, but here are some comments anyway.

Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorder.py Outdated
Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorded_actions.py Outdated
Comment on lines +67 to +68
action = recorder.record(ACTION_TYPE, _params(channel, text), reasoning=reasoning)
return f"Recorded {ACTION_TYPE} (ID: {action['action_id']})."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should file a follow-up issue to centralize the confirmation message.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Filed #6697

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What if we call it actions_manager or something related instead of recorded_actions? WDYT?

@ayoubdiourin7 ayoubdiourin7 Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

actions_manager feels a little generic. What about recorded_actions_manager or action_records? However, I’m also fine with actions_manager.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree. Let's go with action_records.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

BTW, the file name should match to keep things consistent.

Comment thread libs/hackbot-runtime/hackbot_runtime/actions/recorder.py Outdated
@ayoubdiourin7
ayoubdiourin7 requested review from suhaibmujahid and removed request for suhaibmujahid August 25, 2026 09:11
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.

Support managing the reported agent actions

2 participants