Skip to content

feat: add new prompt best practices section #894

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

PeriniM
Copy link
Contributor

@PeriniM PeriniM commented Jul 8, 2025

sequenceDiagram
    participant User
    participant LangSmith
    participant Intermediary Service
    participant GitHub Repo

    User->>LangSmith: Commits new prompt version
    activate LangSmith
    LangSmith->>Intermediary Service: Fires Webhook (sends prompt data)
    deactivate LangSmith
    activate Intermediary Service
    Intermediary Service->>GitHub Repo: Commits new prompt file via API
    activate GitHub Repo
    GitHub Repo-->>Intermediary Service: Confirms commit
    deactivate GitHub Repo
    Intermediary Service-->>LangSmith: Responds with success (optional)
    deactivate Intermediary Service
Loading
gitGraph
    commit id: "v1.0"
    branch develop
    checkout develop
    commit id: "feat: new idea"
    commit id: "fix: refine idea" tag: "dev"
    checkout main
    merge develop id: "Release v2.0" tag: "staging" tag: "prod"
    checkout develop
    commit id: "feat: another feature"
Loading
graph TD
    subgraph Trigger
        A[Commit New Prompt Version];
    end

    subgraph "CI: Automated Testing"
        B(Step 1: Linting & Validation);
        C(Step 2: Unit Tests);
        D(Step 3: Run Evals on Dataset);
    end

    subgraph "CD: Automated Deployment"
        E{Evals Pass & Score Improves?};
        F[Auto-Promote to 'staging'];
        G[Fail & Alert Team];
        H[Manual Review of Staging];
        I[Promote to 'prod'];
    end

    A --> B;
    B --> C;
    C --> D;
    D --> E;
    E -- Yes --> F;
    E -- No --> G;
    F --> H;
    H --> I;
Loading

Copy link

vercel bot commented Jul 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langsmith-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2025 1:08pm

Copy link
Contributor

@katmayb katmayb left a comment

Choose a reason for hiding this comment

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

Hi @PeriniM, Tanushree sent me this PR to take a look through while I ramp up on LangSmith docs. I made some suggestions — I hope these are helpful. Lmk if you have any questions! cc @tanushree-sharma

@katmayb
Copy link
Contributor

katmayb commented Jul 11, 2025

Since this is a best practices guide, maybe adding a link to this new page in the Get Started page (here) would be helpful for users to review when looking at the prompt engineering links.

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