Skip to content

plain-forge rules review and fixing - #17

Open
dusano wants to merge 14 commits into
mainfrom
feat/plain-forge-review
Open

plain-forge rules review and fixing#17
dusano wants to merge 14 commits into
mainfrom
feat/plain-forge-review

Conversation

@dusano

@dusano dusano commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

I went through all the rules in plain-forge and fixed factual issues and added missing information.

Additionally, I've instructed Claude Code with the following prompt for each file in /rules:

Check all the facts in @forge/rules/X.md on real examples.

Claude Code generated examples that tested the rules and come with many suggestions how to improve the rules in plain-forge and are now part of this pull request.

Here is the list of changes and fixes:

  • Adding 'module-structure.md'
  • Clarifying difference between Liquid templates and import and requires module.
  • Fixed a wrong fact in linked-resources.md.
  • Making clear in 'bullet-continutation.md' that one statement per line is preferred convention and not a rule.
  • Removing mentions of 'template' from required-concepts.md because it was introducing confusion.
  • Adding to func-specs.md more details and rationale for the complexity limit.
  • Providing more details in test-reqs.md on test types.
  • More precise information in func-specs.md what 'requires' modules pass on.
  • Removing a statement from impl-reqs.md that is not adding value while it might introduce confusion.
  • A small factual fix for required-concepts.md.

@dusano
dusano requested a review from zanjonke August 15, 2026 12:58
@dusano dusano self-assigned this Aug 15, 2026
@dusano dusano added the documentation Improvements or additions to documentation label Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant