feat: selective reconfigure menu for install.sh

[dasirra]

Summary

Adds a two-level TUI menu to install.sh that activates when .env already exists, letting returning users jump directly to any agent/integration setting instead of walking through the full sequential wizard. Fresh installs are completely unchanged.

Source

Closes #40

What Changed

install.sh

• New entry-point branch: if .env exists, sources all integration modules, runs run_reconfigure_menu + run_reconfigure_summary, then exits — the sequential wizard is never reached
• _integration_status AGENT_ID INTG — returns "configured" / "not configured" based on env tokens or GWS credentials file; defined globally so tests can reference it
• _run_git_identity — extracts Screen 3 git identity prompts into a reusable function; Screen 3 now calls it
• _run_agent_submenu AGENT_ID NAME — per-agent submenu loop with integration status annotations; dispatches to existing module functions; guards run_discord_shared behind a guild-already-set check
• run_reconfigure_menu — top-level agent menu loop; builds id|name map once per invocation (single jq call) to avoid redundant subprocesses
• run_full_agent_setup AGENT_ID — runs full setup for an agent marked "(not installed)"
• run_reconfigure_summary — shows configured agents/integrations, creates runtime dirs, offers make restart
• Removed stale comment ("In reconfigure mode, we'll just show all options") from Screen 2

tests/reconfigure.bats (new)

• 18 tests for _integration_status covering all integration types (discord, github, claude, gws, xurl) plus agent-id uppercasing and unknown integration fallback

Tasks

Task	Status	Notes
Implement reconfigure menu in install.sh	DONE
Add bats tests for _integration_status	DONE	18 tests, all passing
Run make test	DONE	103/103 pass
Code review + fixes	DONE	Fixed unconditional run_discord_shared, duplicate local declaration, stale comment, redundant jq passes, added sync comment in test file

Code Review

Two MUST_FIX issues found and resolved:

1. run_discord_shared was called unconditionally in reconfigure mode — now guarded by [ -z "$(env_get "DISCORD_GUILD")" ] in both _run_agent_submenu and run_full_agent_setup
2. local aid_upper token_val was declared twice in the same function scope in run_reconfigure_menu — removed redundant second declaration

Three SHOULD_FIX issues also resolved: stale comment removed, jq pass count halved via id|name map, sync comment added to test file.

---

Built autonomously by /build