Fix trailing-slash project_name and add force-update test

[xsovad06]

Summary

Address CodeRabbit review findings from PR #189:

• Fix _derive_project_name() to normalize trailing slashes ("owner/" no longer produces empty string)
• Add force=True test path for guideline conflict resolution
• Add trailing-slash edge case test for build_variables()

Test Plan

• make check passes (1596 tests)
• New trailing-slash test validates the fix
• Existing tests unaffected

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f98d418b-fa5a-4bcb-b98d-60855d3fbfc0

📥 Commits

Reviewing files that changed from the base of the PR and between b8b4e41 and 2e209f1.

⛔ Files ignored due to path filters (2)

• .claude/agent-memory/cookbook.md is excluded by !.claude/** and included by none
• .claude/rules/bash-patterns.md is excluded by !.claude/** and included by none

📒 Files selected for processing (2)

• sova/commands/templates.py
• tests/test_commands.py

---

Walkthrough

_derive_project_name in templates.py is updated to strip whitespace and trailing slashes from cfg.github_repo, extract the last path segment when / is present, and fall back to "project" for empty results. Two test additions cover the trailing-slash case and assert force=True behavior in the conflict scenario.

Changes

Project name normalization

Layer / File(s)	Summary
_derive_project_name normalization and test coverage sova/commands/templates.py, tests/test_commands.py	_derive_project_name now trims whitespace, strips trailing slashes, and extracts the last path segment from github_repo, falling back to "project" when the result is empty. New test asserts "owner/" → "owner". test_update_guidelines_detects_conflicts extended to assert force=True clears conflicts and produces exactly one update.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the two main changes: fixing trailing-slash handling in project_name derivation and adding a force-update test.
Description check	✅ Passed	The description covers the summary, type of change (bug fix + tests), and test plan verification, though some checklist items are not explicitly addressed.
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.

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

---

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud