Skip to content

docs(agentic-ci): fix stale generator class name in engine.md#780

Merged
andreatgretel merged 2 commits into
mainfrom
agentic-ci/docs/docs-and-references-20260629-engine-generator-rename
Jun 29, 2026
Merged

docs(agentic-ci): fix stale generator class name in engine.md#780
andreatgretel merged 2 commits into
mainfrom
agentic-ci/docs/docs-and-references-20260629-engine-generator-rename

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Summary

architecture/engine.md referenced a generator class LLMTextColumnGenerator
that no longer exists in the engine package. The concrete generator was renamed
to LLMTextCellGenerator
(packages/data-designer-engine/src/data_designer/engine/column_generators/generators/llm_completion.py:150).

This updates the single stale reference on line 60 so the orientation doc
matches the source. The sibling reference in the same sentence
(SamplerColumnGenerator) was verified to still exist and is left unchanged.

Verification

  • rg "LLMTextColumnGenerator" packages/*/src/ → no matches (class is gone).
  • rg "class LLMTextCellGenerator" packages/*/src/ → single match at the
    stated location, same role.

This is a docs-only, single-line change generated by the daily
docs-and-references audit.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions github-actions Bot requested a review from a team as a code owner June 29, 2026 09:46
@github-actions github-actions Bot added agentic-ci Created by agentic-ci automation agentic-ci/docs-and-references Agentic CI - docs-and-references suite labels Jun 29, 2026
@greptile-apps

greptile-apps Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR corrects a stale class name in architecture/engine.md, updating LLMTextColumnGenerator to LLMTextCellGenerator to match the current source.

  • The old name no longer exists anywhere in the codebase; LLMTextCellGenerator is confirmed at packages/data-designer-engine/src/data_designer/engine/column_generators/generators/llm_completion.py.
  • The sibling reference to SamplerColumnGenerator in the same sentence is still valid and is left untouched.

Confidence Score: 5/5

Documentation-only change replacing a stale class name with the correct one, verified against live source.

The single-line edit corrects a class name in a markdown doc. The old name (LLMTextColumnGenerator) is absent from the entire codebase; the new name (LLMTextCellGenerator) is confirmed in the source. No logic, config, or runtime behavior is touched.

No files require special attention.

Important Files Changed

Filename Overview
architecture/engine.md Single-line doc fix: replaces stale class name LLMTextColumnGenerator with the current LLMTextCellGenerator, verified against the live source.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["architecture/engine.md\n(docs)"] -->|"references"| B["SamplerColumnGenerator\n✅ exists in source"]
    A -->|"used to reference"| C["LLMTextColumnGenerator\n❌ no longer exists"]
    A -->|"now references"| D["LLMTextCellGenerator\n✅ exists in source\n(llm_completion.py:150)"]
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["architecture/engine.md\n(docs)"] -->|"references"| B["SamplerColumnGenerator\n✅ exists in source"]
    A -->|"used to reference"| C["LLMTextColumnGenerator\n❌ no longer exists"]
    A -->|"now references"| D["LLMTextCellGenerator\n✅ exists in source\n(llm_completion.py:150)"]
Loading

Reviews (2): Last reviewed commit: "chore: trigger PR checks" | Re-trigger Greptile

@andreatgretel

Copy link
Copy Markdown
Contributor

/authorize-agentic-ci

@andreatgretel andreatgretel merged commit 7ca9187 into main Jun 29, 2026
61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agentic-ci/docs-and-references Agentic CI - docs-and-references suite agentic-ci Created by agentic-ci automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant