Skip to content

Conversation

@Pouyanpi
Copy link
Collaborator

@Pouyanpi Pouyanpi commented Jan 5, 2026

Description

Related Issue(s)

Checklist

  • I've read the CONTRIBUTING guidelines.
  • I've updated the documentation if applicable.
  • I've added tests if applicable.
  • @mentions of the person or team responsible for reviewing proposed changes.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

Documentation preview

https://nvidia-nemo.github.io/Guardrails/review/pr-1563

@Pouyanpi Pouyanpi changed the title Docs/integrations docs(integrations): reorg langchain and move alignscore and safeguarding guides Jan 5, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 5, 2026

Greptile Summary

This PR reorganizes and enhances integration documentation by separating LangChain and LangGraph sections, and adds two new advanced guides for AlignScore deployment and NVIDIA Blueprint integration.

Key changes:

  • Renamed LangChain integration page to "LangChain Frameworks" and separated LangGraph into its own section
  • Added comprehensive AlignScore deployment guide with step-by-step installation instructions
  • Added NVIDIA AI Virtual Assistant Blueprint integration guide with Docker compose configuration

Issues found:

  • Typo: "assitant" instead of "assistant" in blueprint guide
  • Incorrect notebook path reference in blueprint guide

Confidence Score: 4/5

  • Safe to merge after fixing spelling and path issues
  • Documentation changes are low-risk, but contains two syntax errors (typo and incorrect path) that should be corrected for user experience
  • docs/user-guides/advanced/safeguarding-ai-virtual-assistant-blueprint.md needs typo and path corrections

Important Files Changed

Filename Overview
docs/integration/langchain/index.md Reorganized documentation to separate LangChain and LangGraph sections with updated title
docs/user-guides/advanced/align-score-deployment.md New documentation for AlignScore deployment with step-by-step instructions, minor issues found
docs/user-guides/advanced/safeguarding-ai-virtual-assistant-blueprint.md New NVIDIA Blueprint integration guide with typo and incorrect path reference

Sequence Diagram

sequenceDiagram
    participant User
    participant Docs as Documentation Site
    participant LangChain as LangChain Section
    participant LangGraph as LangGraph Section
    participant AlignScore as AlignScore Guide
    participant Blueprint as Blueprint Guide

    User->>Docs: Navigate to integrations
    Docs->>LangChain: View LangChain Frameworks page
    LangChain-->>User: Display LangChain integration guides
    User->>LangGraph: Navigate to LangGraph section
    LangGraph-->>User: Display LangGraph integration guide
    
    User->>Docs: Navigate to advanced guides
    Docs->>AlignScore: View AlignScore deployment
    AlignScore-->>User: Show installation steps & server setup
    
    User->>Docs: Explore blueprint integration
    Docs->>Blueprint: View NVIDIA Blueprint guide
    Blueprint-->>User: Show Docker setup & notebook reference
Loading

Copy link
Contributor

@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.

Additional Comments (2)

  1. docs/user-guides/advanced/safeguarding-ai-virtual-assistant-blueprint.md, line 28 (link)

    syntax: "assitant" is misspelled - should be "assistant"

  2. docs/user-guides/advanced/safeguarding-ai-virtual-assistant-blueprint.md, line 64 (link)

    syntax: The path to the notebook is incorrect - the actual file is at examples/notebooks/safeguard_ai_virtual_assistant_notebook.ipynb not nemoguardrails/examples/notebooks/...

3 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

@miyoungc miyoungc merged commit 643d915 into develop Jan 7, 2026
4 checks passed
@miyoungc miyoungc deleted the docs/integrations branch January 7, 2026 00:10
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.

3 participants