Skip to content

docs: fix broken Full Flow docs link in README#1782

Open
putramkti wants to merge 1 commit into
MervinPraison:mainfrom
putramkti:docs/fix-broken-flow-link
Open

docs: fix broken Full Flow docs link in README#1782
putramkti wants to merge 1 commit into
MervinPraison:mainfrom
putramkti:docs/fix-broken-flow-link

Conversation

@putramkti
Copy link
Copy Markdown

@putramkti putramkti commented Jun 1, 2026

Summary

Fixes a broken docs link in the README. The link in the PraisonAI Flow quickstart section points to https://docs.praison.ai/docs/concepts/flow, which currently returns 404. The published page lives at /docs/concepts/agentflow.

Verification

$ curl -sL -o /dev/null -w "%{http_code}\n" https://docs.praison.ai/docs/concepts/flow
404
$ curl -sL -o /dev/null -w "%{http_code}\n" https://docs.praison.ai/docs/concepts/agentflow
200

Diff

- > πŸ“– [Full Flow docs](https://docs.praison.ai/docs/concepts/flow) β€” visual agent building, component reference, and deployment
+ > πŸ“– [Full Flow docs](https://docs.praison.ai/docs/concepts/agentflow) β€” visual agent building, component reference, and deployment

Single-character-class fix scoped to one line in README.md. Opening an issue first felt excessive for a broken link, happy to file one if preferred.

Summary by CodeRabbit

  • Documentation
    • Updated Langflow integration documentation link to reference the correct agentflow documentation page.

The link pointed to /docs/concepts/flow which returns 404. The actual
page is published at /docs/concepts/agentflow.
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Fix broken Full Flow documentation link in README

πŸ“ Documentation

Grey Divider

Walkthroughs

Description
β€’ Fix broken documentation link in README
β€’ Update Flow docs URL from /flow to /agentflow
β€’ Resolve 404 error in PraisonAI Flow quickstart section
Diagram
flowchart LR
  A["README.md<br/>PraisonAI Flow section"] -- "Update URL<br/>/flow β†’ /agentflow" --> B["Valid docs link<br/>404 β†’ 200"]

Loading

Grey Divider

File Changes

1. README.md πŸ“ Documentation +1/-1

Fix broken Full Flow docs link URL

β€’ Updated broken documentation link in PraisonAI Flow section
β€’ Changed URL from https://docs.praison.ai/docs/concepts/flow to
 https://docs.praison.ai/docs/concepts/agentflow
β€’ Resolves 404 error and points to correct published documentation page

README.md


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented Jun 1, 2026

Code Review by Qodo

🐞 Bugs (1) πŸ“˜ Rule violations (0)

Grey Divider


Remediation recommended

1. Packaged README link unfixed 🐞 Bug ≑ Correctness
Description
This PR updates the Flow docs link only in the repo-root README.md, but src/praisonai/README.md
(used as the package readme via src/praisonai/pyproject.toml) still points to
https://docs.praison.ai/docs/concepts/flow. As a result, PyPI/package readers will continue to hit
the broken (404) documentation link, so the fix is incomplete.
Code

README.md[385]

Evidence
The repo-root README has been updated to /agentflow, but the src/praisonai package still embeds
the old /flow URL, and its pyproject.toml confirms that src/praisonai/README.md is the readme
used for that package.

README.md[382-386]
src/praisonai/README.md[374-386]
src/praisonai/pyproject.toml[1-6]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The Flow docs URL was fixed in the repo-root `README.md`, but the `src/praisonai` package’s README still contains the old `/docs/concepts/flow` link.

## Issue Context
`src/praisonai/pyproject.toml` specifies `readme = "README.md"`, which refers to `src/praisonai/README.md` for package metadata/readme publishing.

## Fix Focus Areas
- src/praisonai/README.md[385-385]
- src/praisonai/pyproject.toml[1-6]

## Expected change
Replace `https://docs.praison.ai/docs/concepts/flow` with `https://docs.praison.ai/docs/concepts/agentflow` in `src/praisonai/README.md` (matching the root README).

β“˜ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2526c0cc-cade-4f28-b765-91ef740d96c8

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between d5f1114 and 5c8d114.

πŸ“’ Files selected for processing (1)
  • README.md

πŸ“ Walkthrough

Walkthrough

This PR updates a single documentation link in the README's Langflow integration section, changing the "Full Flow docs" reference from the /concepts/flow path to the /concepts/agentflow path.

Changes

Documentation Updates

Layer / File(s) Summary
Langflow integration docs link
README.md
The "Full Flow docs" reference link in the Langflow integration section is updated to point to /concepts/agentflow instead of /concepts/flow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A link hops from flow to agentflow's door,
Where docs now lead where they should explore,
One tiny change, but the path is so clearβ€”
The Langflow guide awaits over here! ✨

πŸš₯ Pre-merge checks | βœ… 5
βœ… Passed checks (5 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 change: fixing a broken documentation link in the README by updating the Full Flow docs URL from /docs/concepts/flow to /docs/concepts/agentflow.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ 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.

@MervinPraison
Copy link
Copy Markdown
Owner

@copilot Do a thorough review of this PR. Read ALL existing reviewer comments above from Qodo, Coderabbit, and Gemini first β€” incorporate their findings.

Review areas:

  1. Bloat check: Are changes minimal and focused? Any unnecessary code or scope creep?
  2. Security: Any hardcoded secrets, unsafe eval/exec, missing input validation?
  3. Performance: Any module-level heavy imports? Hot-path regressions?
  4. Tests: Are tests included? Do they cover the changes adequately?
  5. Backward compat: Any public API changes without deprecation?
  6. Code quality: DRY violations, naming conventions, error handling?
  7. Address reviewer feedback: If Qodo, Coderabbit, or Gemini flagged valid issues, include them in your review
  8. Suggest specific improvements with code examples where possible

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Jun 1, 2026

Greptile Summary

This PR fixes a single broken documentation link in README.md, updating the "Full Flow docs" URL from /docs/concepts/flow (404) to /docs/concepts/agentflow (200).

  • Corrects the Full Flow docs hyperlink in the PraisonAI Flow quickstart section so readers are no longer served a 404 page.

Confidence Score: 5/5

Safe to merge β€” the change is a one-line URL correction in a markdown file with no code impact.

A single broken hyperlink is corrected to a verified live URL; no code, logic, or configuration is touched.

No files require special attention.

Important Files Changed

Filename Overview
README.md Single-line fix updating a broken docs link from /docs/concepts/flow (404) to /docs/concepts/agentflow (200); no logic, structure, or content changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["README.md β€” Full Flow docs link"] -->|"Before (404)"| B["https://docs.praison.ai/docs/concepts/flow"]
    A -->|"After (200)"| C["https://docs.praison.ai/docs/concepts/agentflow"]
    B -->|"HTTP 404"| D["❌ Page Not Found"]
    C -->|"HTTP 200"| E["βœ… AgentFlow Docs"]
Loading

Reviews (1): Last reviewed commit: "docs: fix broken Full Flow docs link in ..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates a documentation link in the README.md file, changing the URL path from 'flow' to 'agentflow'. There are no review comments, and I have no feedback to provide.

@MervinPraison
Copy link
Copy Markdown
Owner

@claude You are the FINAL architecture reviewer. If the branch is under MervinPraison/PraisonAI (not a fork), you are able to make modifications to this branch and push directly. SCOPE: Focus ONLY on Python packages (praisonaiagents, praisonai). Do NOT modify praisonai-rust or praisonai-ts. Read ALL comments above from Gemini, Qodo, CodeRabbit, and Copilot carefully before responding.

Phase 1: Review per AGENTS.md

  1. Protocol-driven: check heavy implementations vs core SDK
  2. Backward compatible: ensure zero feature regressions
  3. Performance: no hot-path regressions

Phase 2: FIX Valid Issues
4. For any VALID bugs or architectural flaws found by Gemini, CodeRabbit, Qodo, Copilot, or any other reviewer: implement the fix
5. Push all code fixes directly to THIS branch (do NOT create a new PR)
6. Comment a summary of exact files modified and what you skipped

Phase 3: Final Verdict
7. If all issues are resolved, approve the PR / close the Issue
8. If blocking issues remain, request changes / leave clear action items

@praisonai-triage-agent
Copy link
Copy Markdown
Contributor

praisonai-triage-agent Bot commented Jun 1, 2026

Claude finished @MervinPraison's task β€”β€” View job


Final Architecture Review βœ…

Phase 1: Architecture Review per AGENTS.md

  • Read AGENTS.md guidelines
  • Review PR against protocol-driven design principles
  • Check backward compatibility
  • Verify performance impact

Phase 2: Fix Valid Issues

  • Review all reviewer feedback (Qodo, CodeRabbit, Greptile)
  • Fix Qodo's identified bug in src/praisonai/README.md
  • Push fixes directly to this branch

Phase 3: Final Verdict

  • Provide final approval/verdict
  • Summary of changes made

βœ… APPROVED - Ready to Merge

Architecture Assessment:

  • βœ… Protocol-driven: N/A (documentation change only)
  • βœ… Backward compatible: No API changes
  • βœ… Performance: Zero impact (documentation only)
  • βœ… Scope: Documentation fixes only - no Python package logic changed

Issues Resolved:

  • βœ… Fixed Qodo's bug: Updated duplicate broken link in src/praisonai/README.md:385
  • βœ… Original PR: Root README.md already fixed
  • βœ… Verification: Both documentation sources now point to working /agentflow URL

Files Modified:

  • src/praisonai/README.md - Fixed broken docs link (commit: 7e8ca38)

What was NOT done:

  • No changes to praisonai-rust or praisonai-ts (per scope instructions)
  • No core SDK logic modifications needed

Final Verdict: This PR now comprehensively fixes the broken documentation link issue across both the repository README and the PyPI package README. All reviewer feedback has been addressed. Safe to merge.


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