chore: move tailscale to experimental config

[steveiliop56]

I am having second thoughts about this integration.

Summary by CodeRabbit

• New Features

  • Tailscale-related configuration has been reorganized under an experimental settings section, including a new default state directory and Docker-specific state path.
  • Updated the sample environment variables to match the new experimental Tailscale key names.

• Bug Fixes

  • Tailscale listener/serving activation now depends on the experimental Tailscale enable setting.
  • Funnel behavior and URL/cookie updates now follow the experimental Tailscale configuration.

[coderabbitai]

📝 Walkthrough

Walkthrough

The Tailscale configuration is moved from Config.Tailscale to Config.Experimental.Tailscale. Default initialization, Docker overrides, bootstrap/service call sites, and example environment variables now use the nested experimental config.

Changes

Tailscale config relocation

Layer / File(s)	Summary
Config struct and defaults internal/model/config.go	Config.Tailscale is replaced by Config.Experimental, ExperimentalConfig now contains Tailscale TailscaleConfig, and default/Docker initialization uses Experimental.Tailscale.Dir.
Bootstrap and service consumers updated internal/bootstrap/app_bootstrap.go, internal/bootstrap/router_bootstrap.go, internal/service/tailscale_service.go	App URL replacement, listener selection, service startup, funnel warnings, and funnel listener selection now read Tailscale settings from Config.Experimental.Tailscale.
Example env keys updated .env.example	The example environment file replaces the top-level Tailscale block with TINYAUTH_EXPERIMENTAL_TAILSCALE_* variables.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: scottmckendry, Rycochet

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: moving Tailscale configuration under the experimental config namespace.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/tailscale-exp

---

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.}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/service/tailscale_service.go	0.00%	7 Missing ⚠️
internal/model/config.go	0.00%	5 Missing ⚠️
internal/bootstrap/router_bootstrap.go	0.00%	2 Missing ⚠️
internal/bootstrap/app_bootstrap.go	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@internal/bootstrap/router_bootstrap.go`:
- Around line 133-136: The error message in router_bootstrap.go still uses the
old flat config key even though the check is now in
app.config.Experimental.Tailscale.Listen. Update the fmt.Errorf text in the
tailscaleService nil guard inside the router bootstrap logic to reference the
new Experimental.Tailscale.Listen path, keeping the rest of the validation flow
unchanged.

🪄 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: be45e69c-0a2a-49ab-917c-f9df3a1f3b46

📥 Commits

Reviewing files that changed from the base of the PR and between c8b31c5 and eea7c65.

📒 Files selected for processing (4)

• internal/bootstrap/app_bootstrap.go
• internal/bootstrap/router_bootstrap.go
• internal/model/config.go
• internal/service/tailscale_service.go

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.env.example:
- Around line 224-239: The experimental env block in .env.example needs to be
normalized to satisfy dotenv-linter. Reorder the
TINYAUTH_EXPERIMENTAL_TAILSCALE_* entries to match the repo’s existing
convention, and update the TINYAUTH_EXPERIMENTAL_TAILSCALE_DIR default in the
experimental config section to remove the unnecessary quotes. Make the changes
within the experimental config block so the regenerated example remains
lint-clean.

🪄 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 483a636d-9076-4e78-a51e-89302b8e8c7e

📥 Commits

Reviewing files that changed from the base of the PR and between 2f8dee0 and b1ae78f.

📒 Files selected for processing (1)

• .env.example