docs: rewrite AGENTS.md as a concise agent operating guide#307
Open
johnnygreco wants to merge 3 commits intomainfrom
Open
docs: rewrite AGENTS.md as a concise agent operating guide#307johnnygreco wants to merge 3 commits intomainfrom
johnnygreco wants to merge 3 commits intomainfrom
Conversation
Streamline from verbose reference doc to focused operating guide with architecture diagram, first-5-minutes checklist, change strategy, and quality gate sections. Remove redundant code style examples already enforced by ruff.
- Add step to checklist for picking make targets before raw uv commands - Add quick make command map for common workflows - Expand testing section with per-package targets and iteration tips - Add make verify-imports to useful commands - Refine quality gate with license header and check-all reminders Co-authored-by: Cursor <cursoragent@cursor.com>
Greptile OverviewGreptile SummaryComplete rewrite of
The rewrite removes verbose examples and detailed listings (PEP 8 naming examples, testing patterns with code samples, pre-commit hooks documentation, active linter rules listing) in favor of concise, actionable guidance. All technical references to file paths,
|
| Filename | Overview |
|---|---|
| AGENTS.md | rewrites documentation as concise agent guide with architecture diagram, checklists, and streamlined workflow - missing two column config types from list |
Sequence Diagram
sequenceDiagram
participant User as User/App Code
participant Builder as DataDesignerConfigBuilder
participant Interface as DataDesigner Interface
participant ResourceProvider as ResourceProvider
participant Compiler as compile_data_designer_config
participant ColumnBuilder as ColumnWiseDatasetBuilder
participant DAG as DAG Sorter
participant Registry as DataDesignerRegistry
participant Generators as Column Generators
participant BatchManager as DatasetBatchManager
participant Processors as Processors
participant Storage as ArtifactStorage
participant Profiler as DatasetProfiler
User->>Builder: create config (add_column calls)
Builder->>Builder: build() → DataDesignerConfig
User->>Interface: generate() or validate()
Interface->>ResourceProvider: create_resource_provider()
ResourceProvider->>ResourceProvider: initialize ModelRegistry, MCPRegistry, SeedReaderRegistry, ArtifactStorage
Interface->>Compiler: compile_data_designer_config()
Compiler->>ColumnBuilder: create ColumnWiseDatasetBuilder
ColumnBuilder->>DAG: sort columns by dependencies
DAG->>Registry: resolve generators/validators from config
Registry->>Generators: instantiate column generators
loop For each batch
Generators->>BatchManager: generate column data
BatchManager->>Processors: apply processors
Processors->>Storage: store artifacts
end
Storage->>Profiler: profile dataset
Profiler->>Interface: return results
Interface->>User: return DataDesignerResult
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Summary
Rewrites
AGENTS.mdfrom a verbose ~574-line reference document into a concise ~230-line operating guide, optimized for agent consumption with clear priorities, actionable checklists, and a visual architecture diagram.🔄 Changes
🔧 Changed
AGENTS.md: Complete rewrite with the following structural improvements:maketargets🗑️ Removed
🤖 Generated with AI
Made with Cursor