Skip to content

docs(tools): add Lians agent memory integration#6584

Open
ebeirne wants to merge 1 commit into
crewAIInc:mainfrom
ebeirne:docs/lians-agent-memory
Open

docs(tools): add Lians agent memory integration#6584
ebeirne wants to merge 1 commit into
crewAIInc:mainfrom
ebeirne:docs/lians-agent-memory

Conversation

@ebeirne

@ebeirne ebeirne commented Jul 17, 2026

Copy link
Copy Markdown

Summary

  • document the Lians CrewAI memory tools
  • show zero-setup local storage and agent wiring
  • list the four available memory operations

Validation

  • checked the complete changed files for Unicode em dashes
  • checked the diff for whitespace errors

AI assistance disclosure

This contribution was generated with AI assistance and reviewed before submission. Per the contribution policy, this PR requires the llm-generated label. GitHub does not allow this fork author to apply labels to the upstream repository, so a maintainer or repository automation must add it.

@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds documentation for Lians Agent Memory Tools with local and remote CrewAI usage, lists supported memory operations, links the page from the integration overview, and revises latency guidance wording.

Changes

Lians integration documentation

Layer / File(s) Summary
Lians memory tools page
docs/edge/en/tools/integration/lians.mdx
Documents installation, local and remote client setup, CrewAI usage, available memory tools, and stable agent_id usage.
Integration overview updates
docs/edge/en/tools/integration/overview.mdx
Adds the Lians tools card and updates the latency best-practice wording while retaining its polling and timeout guidance.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: adding Lians agent memory integration docs.
Description check ✅ Passed The description accurately summarizes the Lians memory docs, local storage, wiring, and operations.
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.
✨ 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.

@ebeirne
ebeirne marked this pull request as ready for review July 18, 2026 01:14

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
docs/edge/en/tools/integration/lians.mdx (1)

60-64: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Include an actionable remote-client example.

This section currently tells readers to create the standard client but does not show the constructor or credential wiring. Add the documented LiansClient(base_url=..., api_key=...) example so remote setup is directly usable. (pypi.org)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/edge/en/tools/integration/lians.mdx` around lines 60 - 64, Update the
“Remote service” section to include an actionable LiansClient example showing
base_url and api_key credentials, then demonstrate passing that client to
build_crewai_tools instead of LocalLiansClient. Preserve the existing link to
the Lians documentation.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/edge/en/tools/integration/lians.mdx`:
- Line 16: Update the installation command in the lians integration
documentation to list crewai and lians-sdk as separate pip arguments, keeping
the extras specifier attached to lians-sdk.

---

Nitpick comments:
In `@docs/edge/en/tools/integration/lians.mdx`:
- Around line 60-64: Update the “Remote service” section to include an
actionable LiansClient example showing base_url and api_key credentials, then
demonstrate passing that client to build_crewai_tools instead of
LocalLiansClient. Preserve the existing link to the Lians documentation.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3572a02d-10e4-46b8-9768-fff014733120

📥 Commits

Reviewing files that changed from the base of the PR and between 69c0308 and b337b0e.

📒 Files selected for processing (2)
  • docs/edge/en/tools/integration/lians.mdx
  • docs/edge/en/tools/integration/overview.mdx

Install CrewAI and the Lians CrewAI integration. Include the `local` extra for zero-setup local storage.

```bash
pip install "crewai lians-sdk[crewai,local]"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n## File context\n'
sed -n '1,120p' docs/edge/en/tools/integration/lians.mdx | cat -n

printf '\n## Search for related install commands\n'
rg -n "pip install|lians-sdk|crewai" docs/edge/en/tools/integration/lians.mdx

Repository: crewAIInc/crewAI

Length of output: 3184


🏁 Script executed:

python3 - <<'PY'
import sys
req = 'crewai lians-sdk[crewai,local]'
try:
    import packaging.requirements as pr
    parsed = pr.Requirement(req)
    print("parsed:", parsed)
except Exception as e:
    print("error:", type(e).__name__, e)

tests = ['crewai', 'lians-sdk[crewai,local]', 'crewai lians-sdk[crewai,local]']
for t in tests:
    try:
        parsed = pr.Requirement(t)
        print(t, "=> OK", parsed)
    except Exception as e:
        print(t, "=>", type(e).__name__, e)
PY

Repository: crewAIInc/crewAI

Length of output: 543


Fix the install command

pip install "crewai lians-sdk[crewai,local]" is invalid requirement syntax. Split it into separate packages: pip install crewai "lians-sdk[crewai,local]".

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/edge/en/tools/integration/lians.mdx` at line 16, Update the installation
command in the lians integration documentation to list crewai and lians-sdk as
separate pip arguments, keeping the extras specifier attached to lians-sdk.

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