Skip to content

chore: update architecture.md and AGENTS.md#617

Merged
fansenze merged 1 commit intomainfrom
chore/architecture
Apr 8, 2026
Merged

chore: update architecture.md and AGENTS.md#617
fansenze merged 1 commit intomainfrom
chore/architecture

Conversation

@simplyme0823
Copy link
Copy Markdown
Contributor

Summary

update architecture.md and AGENTS.md

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request provides a comprehensive update to the project's architectural documentation, aligning it with the current system design. Key changes include updated diagrams for the system and data flow, a more detailed directory map, and refined descriptions of the parsing pipeline, configuration merging, and rule framework. The review feedback correctly identifies a minor inaccuracy regarding the rule registration location and suggests improving the new rule template by re-adding an example for parsing rule-specific options.

Comment thread architecture.md Outdated
Comment thread architecture.md Outdated
@simplyme0823 simplyme0823 force-pushed the chore/architecture branch 3 times, most recently from 3f359fd to 0c5024b Compare April 8, 2026 09:46
@fansenze fansenze merged commit c80080c into main Apr 8, 2026
10 checks passed
@fansenze fansenze deleted the chore/architecture branch April 8, 2026 10:20
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