Skip to content

Conversation

@vim-zz
Copy link
Collaborator

@vim-zz vim-zz commented Nov 25, 2025

image

✨ PR Description

Purpose: Add documentation for code review action outputs and demonstrate usage via context variables in automation workflows.
Main changes:

  • Documented code-review action output parameters with the new is_LGTM boolean field
  • Added examples of conditional automation workflows using action output context variables
  • Created new 'actions' context variable reference section with syntax guidelines and usage examples

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

@vim-zz vim-zz requested a review from MishaKav November 25, 2025 11:44
Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

Copy link
Contributor

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

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

✨ PR Review

LGTM

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how

@gitstream-cm gitstream-cm bot requested a review from a team November 25, 2025 11:48
gitstream-cm[bot]
gitstream-cm bot previously approved these changes Nov 25, 2025
Copy link
Collaborator

@MishaKav MishaKav left a comment

Choose a reason for hiding this comment

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

i think we should add some note, that if the automation name with - and not _, so the syntax shoudl be changed a little

{{ actions.["ai-code-review"].outputs.is_LGTM }}

@gitstream-cm gitstream-cm bot requested a review from a team November 25, 2025 12:23
@gitstream-cm gitstream-cm bot dismissed their stale review November 25, 2025 12:23

Review dismissed

@vim-zz
Copy link
Collaborator Author

vim-zz commented Nov 25, 2025

@MishaKav is it {{ actions["ai-code-review"].outputs.is_LGTM }} without the dot?

Copy link
Contributor

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

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

✨ PR Review

The PR adds comprehensive documentation for action outputs and usage examples. The documentation is functional and provides clear guidance on using the is_LGTM output with helpful examples.

1 issues detected:

🧾 Readability - The same format specification appears twice within a few lines, reducing documentation clarity. 🛠️

Details: The format for accessing action outputs (actions.<automation_id>.outputs.<output_name>) is stated twice in lines 305 and 307, making the documentation unnecessarily redundant and less concise for readers.
File: docs/automation-actions.md (307-307)
🛠️ A suggested code correction is included in the review comments.

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how


Use the `actions` context variable to access outputs from this action in subsequent automations. The format is `actions.<automation_id>.outputs.<output_name>`, where `<automation_id>` is the identifier of the automation that ran the action.

Use the syntax: `actions.<automation_id>.outputs.<output_name>`
Copy link
Contributor

Choose a reason for hiding this comment

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

🧾 Readability - Redundant Format: Remove line 307 as the format specification is already provided in line 305. The detailed component explanation in lines 309-310 can follow directly after the initial format introduction.

Suggested change
Use the syntax: `actions.<automation_id>.outputs.<output_name>`
Is this review accurate? Use 👍 or 👎 to rate it

If you want to tell us more, use /gs feedback e.g. /gs feedback this review doesn't make sense, I disagree, and it keeps repeating over and over

Copy link
Contributor

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

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

✨ PR Review

LGTM

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how

@MishaKav
Copy link
Collaborator

@MishaKav is it {{ actions["ai-code-review"].outputs.is_LGTM }} without the dot?
Yes, this is the jinja2 syntax

image

Copy link
Collaborator

@MishaKav MishaKav left a comment

Choose a reason for hiding this comment

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

LG

@gitstream-cm gitstream-cm bot requested a review from a team November 25, 2025 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants