Skip to content

update github action docs to match ambient-action v2#1244

Open
vishsanghishetty wants to merge 1 commit intoambient-code:mainfrom
vishsanghishetty:docs/1232-update-github-action-docs
Open

update github action docs to match ambient-action v2#1244
vishsanghishetty wants to merge 1 commit intoambient-code:mainfrom
vishsanghishetty:docs/1232-update-github-action-docs

Conversation

@vishsanghishetty
Copy link
Copy Markdown
Contributor

@vishsanghishetty vishsanghishetty commented Apr 7, 2026

Closes #1232


Why

The GitHub Action docs were pinned to v0.0.2 and hadn't been updated since. Several inputs, an output, and the timeout semantics have all changed in ambient-code/ambient-action — the docs didn't reflect any of it.

What changed

Inputs table

Change Detail
Added session-name Send a message to an existing session instead of creating a new one
Added stop-on-run-finished Auto-stop session when the agent finishes its run
Added poll-timeout Max minutes to poll before giving up (when wait: true)
Fixed timeout Was "session timeout in minutes, default 30" → now "inactivity timeout in seconds, default 0" (matches action.yml)
Updated descriptions api-token, project, prompt, wait, poll-interval — aligned wording with action.yml

Outputs table

Change Detail
Added session-url URL to the session in the Ambient UI

Version refs

  • All 4 occurrences of @v0.0.2@v2 (across github-action.md and bugfix.md)
  • Removed hardcoded (v0.0.2) from the intro paragraph

Files touched

File What
docs/src/content/docs/extensions/github-action.md Inputs/outputs tables, version refs, descriptions
docs/src/content/docs/workflows/bugfix.md Version ref in YAML example

How I tested

Compared every input and output in the updated docs against action.yml line by line — all 17 inputs and 5 outputs match. No code changes, so no lint/test/tsc applicable. Astro build will run in CI (couldn't build locally — needs Node 22+, local is Node 20).

Summary by CodeRabbit

  • Documentation
    • Clarified GitHub Action inputs and outputs wording for parameters like API, project, timeout, wait, and poll settings to improve comprehension.
    • Reordered related settings in the docs for clearer presentation.
    • Adjusted session output phrasing to emphasize created session name and UID.
    • Updated all example workflows to reference the action v2.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d8f680fa-8d5c-4951-9832-64975199cf27

📥 Commits

Reviewing files that changed from the base of the PR and between 205b873 and c9d5ed7.

📒 Files selected for processing (2)
  • docs/src/content/docs/extensions/github-action.md
  • docs/src/content/docs/workflows/bugfix.md
✅ Files skipped from review due to trivial changes (2)
  • docs/src/content/docs/workflows/bugfix.md
  • docs/src/content/docs/extensions/github-action.md

📝 Walkthrough

Walkthrough

Updated GitHub Action documentation and example workflows: clarified input/output descriptions and wording for several action inputs (api-url, api-token, project, timeout, wait, poll-*), reordered parameter rows, and bumped example action references from ambient-code/ambient-action@v0.0.5 to ambient-code/ambient-action@v2.

Changes

Cohort / File(s) Summary
GitHub Action docs
docs/src/content/docs/extensions/github-action.md
Reworded input descriptions for api-url, api-token, project, timeout, wait, poll-interval, poll-timeout; reordered parameter table rows; updated outputs session-name/session-uid phrasing; bumped example action reference to ambient-code/ambient-action@v2.
Workflow examples
docs/src/content/docs/workflows/bugfix.md
Updated example step to use ambient-code/ambient-action@v2 (no other YAML changes).
🚥 Pre-merge checks | ✅ 7 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning Title lacks Conventional Commits format prefix (type required: feat, fix, chore, docs, refactor, test, ci, perf). Reformat title as 'docs: update github action docs to match ambient-action v2' to comply with Conventional Commits.
✅ Passed checks (7 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed Changes fully satisfy #1232 requirements: updated GitHub Action docs to match ambient-action v2 with revised input/output descriptions and version references.
Out of Scope Changes check ✅ Passed All changes are documentation-only updates directly addressing #1232; no extraneous modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Performance And Algorithmic Complexity ✅ Passed Pull request modifies only markdown documentation with no code changes, version bumps in YAML examples, and zero performance implications.
Security And Secret Handling ✅ Passed Documentation changes are security-compliant with proper GitHub secrets usage (${{ secrets.ACP_TOKEN }}, ${{ secrets.ACP_URL }}) and bearer token authentication; no hardcoded credentials or injection vulnerabilities.
Kubernetes Resource Safety ✅ Passed Kubernetes Resource Safety check not applicable; PR contains only documentation updates to GitHub Action reference pages with no K8s resource definitions or deployment configurations present.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@vishsanghishetty vishsanghishetty force-pushed the docs/1232-update-github-action-docs branch from 5b2c611 to 205b873 Compare April 10, 2026 19:48
@vishsanghishetty
Copy link
Copy Markdown
Contributor Author

@ambient-code

the docs were pinned to v0.0.2 and missing several inputs/outputs
that have been added since. syncs the inputs table (adds session-name,
stop-on-run-finished, poll-timeout), outputs table (adds session-url),
fixes the timeout description, and bumps all examples to @v2.

closes ambient-code#1232

Signed-off-by: Vishali <vsanghis@redhat.com>
@vishsanghishetty vishsanghishetty force-pushed the docs/1232-update-github-action-docs branch from 205b873 to c9d5ed7 Compare April 12, 2026 12:01
@vishsanghishetty
Copy link
Copy Markdown
Contributor Author

@ambient-code

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.

Our Docs our out of date for the Github action

1 participant