Skip to content

style: Fix formatting of user facing and agent facing diagrams and markdown tables#892

Merged
TabishB merged 3 commits intoFission-AI:mainfrom
kunzaatko:fix-formatting-explore
Apr 9, 2026
Merged

style: Fix formatting of user facing and agent facing diagrams and markdown tables#892
TabishB merged 3 commits intoFission-AI:mainfrom
kunzaatko:fix-formatting-explore

Conversation

@kunzaatko
Copy link
Copy Markdown
Contributor

@kunzaatko kunzaatko commented Mar 30, 2026

Summary by CodeRabbit

  • Style
    • Improved formatting and alignment of workflow documentation: updated tables, diagrams, and command reference layouts for onboarding, exploration, and bulk-archive workflows to enhance readability and visual consistency.
    • Minor spacing and indentation adjustments across embedded instructions and rendered documentation for clearer presentation.

@kunzaatko kunzaatko requested a review from TabishB as a code owner March 30, 2026 21:42
Copy link
Copy Markdown

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 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: 589f1563-3df7-4f6f-876c-582725bf955b

📥 Commits

Reviewing files that changed from the base of the PR and between d8d93cb and 0dac892.

📒 Files selected for processing (1)
  • test/core/templates/skill-templates-parity.test.ts

📝 Walkthrough

Walkthrough

Three workflow template files had only presentational edits to embedded Markdown/ASCII docs (spacing, table column alignment, and indentation). Tests updated expected SHA-256 hashes to match the changed template outputs. No runtime logic, exports, or APIs were modified.

Changes

Cohort / File(s) Summary
Workflow Template Formatting
src/core/templates/workflows/bulk-archive-change.ts, src/core/templates/workflows/explore.ts, src/core/templates/workflows/onboard.ts
Adjusted spacing, padding, column alignment, and ASCII-diagram indentation inside embedded instruction/template strings; purely presentational changes only.
Tests: Parity Hashes
test/core/templates/skill-templates-parity.test.ts
Updated expected SHA-256 hash constants for generated template factory outputs and generated skill content to reflect the textual/formatting changes in the templates.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I nibbled at spaces, nudged tables in line,

ASCII boxes centered, headers now fine,
Templates hum tidy with hashes anew,
No logic disturbed — just a neat rendezvous.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: formatting fixes to diagrams and markdown tables across multiple template files, which aligns with the actual modifications made to workflow templates and their corresponding test hashes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

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

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.

TabishB
TabishB previously approved these changes Apr 9, 2026
Copy link
Copy Markdown
Contributor

@TabishB TabishB left a comment

Choose a reason for hiding this comment

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

thank you!

@TabishB TabishB enabled auto-merge April 9, 2026 03:14
@TabishB
Copy link
Copy Markdown
Contributor

TabishB commented Apr 9, 2026

This will require a test update to fix,

@TabishB
Copy link
Copy Markdown
Contributor

TabishB commented Apr 9, 2026

@1code fix the related failing tests in this PR

@1code-async
Copy link
Copy Markdown
Contributor

1code-async bot commented Apr 9, 2026

👋 Hey @TabishB! I'm on it.

Task: fix the related failing tests in this PR

I'll update this comment when complete.


View live progress

Powered by 1Code

@TabishB TabishB added this pull request to the merge queue Apr 9, 2026
Merged via the queue into Fission-AI:main with commit 7fd5417 Apr 9, 2026
9 checks passed
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.

2 participants