feat: increment suggested module tags

[ViTeXFTW]

Suggest next numeric ModuleTag completion; ignores descriptive tags and sub-block arguments. Closes #108.

[greptile-apps]

Greptile Summary

The PR changes module-name completion snippets to derive the next numeric module tag from tags already present in the enclosing scope.

• Carries the enclosing syntax scope through module-name completion classification.
• Filters existing tags to genuine module slots while ignoring descriptive tags and sub-block arguments.
• Adds tests for numeric sequencing, case-insensitive prefixes, and ignored descriptive tags.

Confidence Score: 4/5

The nested-module tag collision should be fixed before merging because completion can insert a tag already used elsewhere in the same object.

The new search considers only direct object children even though valid tagged modules and uniqueness enforcement span nested reentrant sub-blocks; it also repeats the maximum u32 tag at the numeric boundary.

Files Needing Attention: crates/analysis/src/completion.rs

Important Files Changed

Filename	Overview
crates/analysis/src/completion.rs	Adds scope-aware numeric module-tag suggestions, but direct-child traversal misses tagged modules in reentrant sub-blocks and the u32 boundary can repeat an existing tag.

_{Reviews (1): Last reviewed commit: "feat: increment suggested module tags" | Re-trigger Greptile}

[greptile-apps]

Nested module tags are skipped

When a reentrant sub-block such as AddModule or ReplaceModule contains a numeric module tag, scanning only the object's direct children excludes that tag. Completion can therefore insert the same tag for a direct module, causing a duplicate-module-tag diagnostic and making tag-based module removal ambiguous.

Knowledge Base Used: Completion and code actions

[greptile-apps]

Saturating increment repeats maximum tag

With an existing ModuleTag_4294967295, saturating_add(1) returns the same value, so completion suggests an already-used tag and produces an avoidable duplicate-tag diagnostic.

Knowledge Base Used: Completion and code actions

[ViTeXFTW]

Addressed the review: completion now scans genuine module slots throughout the enclosing Object, including re-entrant AddModule scopes, and supports values past u32::MAX. Added regression tests for both.