Skip to content

feat(flows): include the flow url in flows list --remote --json and document the flow code branch - #1535

Merged
Mateus Zitelli (MateusZitelli) merged 1 commit into
mainfrom
mateus/flows-list-remote-url
Aug 28, 2026
Merged

feat(flows): include the flow url in flows list --remote --json and document the flow code branch#1535
Mateus Zitelli (MateusZitelli) merged 1 commit into
mainfrom
mateus/flows-list-remote-url

Conversation

@MateusZitelli

Copy link
Copy Markdown
Contributor

Relates to qawolf/platform#32412 and qawolf/platform#32418

Overview of Changes

flows list --remote --json now includes each flow's url (the flow page on the platform), which flow.list already returns but the CLI dropped; agents were guessing app routes with curl after they had the id. The agent skill's "Git-backed workflows" section now explains that the platform reads flows from the environment's flow code branch (environment getflowCodeBranch), that a flow gets a page only once that branch has reconciled the commit, and how to land a flow there and read its url. Human and agent tables are unchanged; a minor changeset is included.

Testing

bun run typecheck
bun run lint
bun run format:check
bun run knip
bun run test
bun run build

listRemote tests assert the url field in every JSON expectation; the skill markdown regenerates through bun run generate.

Checklist

  • Changes follow the code style of this project
  • Self-review completed
  • Tests added/updated (or not applicable)
  • No breaking changes (or described below)

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 96fc00aa-e8b6-47ce-b7d5-928cf388c468

📥 Commits

Reviewing files that changed from the base of the PR and between 0eadd54 and b19e8b6.

📒 Files selected for processing (5)
  • .changeset/flows-list-remote-url.md
  • skills/qawolf-cli/SKILL.md
  • src/commands/qawolfCliSkill.template.md
  • src/domains/flows/listRemote.test.ts
  • src/domains/flows/listRemote.ts

Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 4 reviews per hour.


Walkthrough

Remote flow list items now include each flow’s API-provided platform URL. Tests cover standard, filtered, and ad-hoc execution-target listings. CLI skill guidance now documents publishing flow code to the configured branch, verifying synchronization, and retrieving the flow ID and platform URL. A changeset records the CLI update.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to b19e8

This localized change exposes flow URLs in remote JSON output and updates related documentation and tests; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: chajac, gorangajic, theonly1me

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title uses a valid Conventional Commit format and accurately describes the changes, but it is 97 characters long and exceeds the 72-character limit. Shorten the title to 72 characters or fewer while preserving the main change, for example: "feat(flows): include remote flow URLs and document flow code branch"
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description follows the required template. It includes related issues, an overview, testing commands, implementation-specific test details, and a completed checklist.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mateus/flows-list-remote-url

Comment @coderabbitai help to get the list of available commands.

@MateusZitelli
Mateus Zitelli (MateusZitelli) merged commit 5626e2c into main Aug 28, 2026
7 checks passed
@MateusZitelli
Mateus Zitelli (MateusZitelli) deleted the mateus/flows-list-remote-url branch August 28, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants