Add regex_replace_chars to main.tf

[goruha]

what

• Allow repo name to contain dots

why

• Normalized null-label ID doesn't allow IDs including dot character. Effectively, it tries to change repository name like .github -> github.

references

• fix: pass variable as a name cloudposse/terraform-github-repository#24

Summary by CodeRabbit

• Configuration Updates
  • Repository configuration now includes a new character-pattern parameter to improve validation and handling of allowed/removed characters in repository names and related identifiers, reducing invalid-name errors and improving consistency across tooling and displays.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 36894b42-1c85-4791-9e13-8295cd488a74

📥 Commits

Reviewing files that changed from the base of the PR and between 47f8101 and a83f341.

📒 Files selected for processing (1)

• src/main.tf

✅ Files skipped from review due to trivial changes (1)

• src/main.tf

---

📝 Walkthrough

Walkthrough

The PR adds a single input argument, regex_replace_chars = "/[^a-zA-Z0-9.-]/", to module "repository" in src/main.tf. No other logic, locals, or control flow were modified.

Changes

Cohort / File(s)	Summary
Terraform module configuration src/main.tf	Added regex_replace_chars = "/[^a-zA-Z0-9.-]/" input to module "repository"; no other edits in the file.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibble patterns in the night,
Tidy dots, dashes, letters—just right,
A tiny regex tucked in place,
The repo hops with tidier grace.

🚥 Pre-merge checks | ✅ 3

✅ 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 and clearly describes the main change: adding the regex_replace_chars parameter to main.tf.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch allow-repo-name-contains-dots

---

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: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/main.tf`:
- Line 13: The regex in the variable regex_replace_chars currently uses
"/[^.-a-zA-Z0-9]/" which accidentally defines a range from '.' to 'a'; update
the character class so the hyphen is treated literally by placing it at the end
(e.g., "/[^.a-zA-Z0-9-]/") so only dot, hyphen and alphanumeric characters are
allowed; modify the value of regex_replace_chars accordingly, keeping the
surrounding quotes and slashes intact.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: af1115f6-6289-4ce1-b345-45552e91560c

📥 Commits

Reviewing files that changed from the base of the PR and between ed122dc and bf1cd65.

📒 Files selected for processing (1)

• src/main.tf

[github-actions]

These changes were released in v1.1.0.