Refactor DummyExpression class and improve method summaries

[arika0093]

Refactor the DummyExpression class for clarity and update the summaries of the SelectExpr methods to better describe their functionality.

close #34
close #37

Summary by CodeRabbit

• Refactor
  • Internal code restructuring to reorganize source generation components. No user-facing changes.

[coderabbitai]

Walkthrough

Restructures the DummyExpression helper class by moving it from a nested public class within the Linqraft namespace to a top-level internal class. The four SelectExpr method overloads remain functionally identical but transition to internal visibility, reducing the public API surface.

Changes

Cohort / File(s)	Summary
Dummy Expression Restructuring src/Linqraft.SourceGenerator/ConstSourceCodes.cs	Migrates DummyExpression class from nested public to top-level internal scope. Preserves four SelectExpr overloads with updated XML documentation. Method bodies unchanged (continue throwing NotImplementedException).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Single file modification with straightforward structural reorganization
• No functional logic changes; methods retain identical behavior
• Accessibility scope reduction only (public → internal)

Possibly related issues

• bug: DummyExpression.SelectExpr should be internal #34: The conversion of DummyExpression to internal directly addresses the requirement that DummyExpression.SelectExpr should be internal rather than public.

Poem

A rabbit hops, organizing its burrow with care, 🐰
Moving the dummy from the public square,
Now internal and tidy, at the top of the nest,
Where helper methods rest—refined for the best!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title directly reflects the main changes: refactoring the DummyExpression class structure and improving method documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/global_internal_selectexpr

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (4)

src/Linqraft.SourceGenerator/ConstSourceCodes.cs (4)

48-50: Improve grammar in XML summary.

The summary has grammatical issues that affect clarity:

• "Create select expression method" lacks a subject (should be "Creates a select expression method")
• "usable nullable operators" is awkward (consider "enables nullable operators" or "supports nullable operators")

Apply this diff to improve readability:

 /// <summary>
-/// Create select expression method, usable nullable operators, and generate instance DTOs.
+/// Creates a select expression method that enables nullable operators and generates instance DTOs.
 /// </summary>

---

57-59: Same grammar issue in XML summary.

The summary has the same grammatical issues as the previous overload. Consider applying the same fix for consistency.

Apply this diff:

 /// <summary>
-/// Create select expression method, usable nullable operators, and generate instance DTOs.
+/// Creates a select expression method that enables nullable operators and generates instance DTOs.
 /// </summary>

---

67-70: Fix grammatical error in XML summary.

The summary has grammatical errors:

• "Create select expression method" lacks proper subject/verb agreement
• "with generate instance DTOs" should be "and generates instance DTOs"

Apply this diff:

 /// <summary>
-/// Create select expression method, with generate instance DTOs.
+/// Creates a select expression method and generates instance DTOs.
 /// Works with IEnumerable where nullable operators are supported natively.
 /// </summary>

---

77-80: Same grammatical error as previous IEnumerable overload.

Apply the same grammatical fix for consistency.

Apply this diff:

 /// <summary>
-/// Create select expression method, with generate instance DTOs.
+/// Creates a select expression method and generates instance DTOs.
 /// Works with IEnumerable where nullable operators are supported natively.
 /// </summary>

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cf3e820 and f18bf10.

📒 Files selected for processing (1)

• src/Linqraft.SourceGenerator/ConstSourceCodes.cs (1 hunks)

🔇 Additional comments (1)

src/Linqraft.SourceGenerator/ConstSourceCodes.cs (1)

43-46: LGTM: Class declaration and accessibility change are appropriate.

The transition from a nested public class to a top-level internal class correctly restricts the API surface for source generator scaffolding. The summary clearly indicates these are placeholder methods.