Fix #437: add 10 editsite flags for tabcmd Classic parity

[jacalata]

Summary

tabcmd Classic exposes 12 site-configuration flags on editsite that tabcmd 2 was missing. This adds 10 of them. The 2 skipped:

• --metrics-level — Tableau-internal VizQL telemetry, likely Cloud-N/A (pending VizQL team confirmation)
• --cache-warmup-threshold — server-only, not applicable to Cloud

Flags added, with the SiteItem attribute each wires:

Classic flag	SiteItem attribute
--guest-access-enabled / -g	guest_access_enabled
--cache-warmup / --no-cache-warmup	cache_warmup_enabled
--subscription-email / -e	custom_subscription_email + _enabled
--subscription-footer / -f	custom_subscription_footer + _enabled
--web-extraction-enabled	web_extraction_enabled
--allow-subscriptions / --no-allow-subscriptions	disable_subscriptions (inverted)
--allow-web-authoring / --no-allow-web-authoring	authoring_enabled
--allow-mobile-snapshots / --no-allow-mobile-snapshots	sheet_image_enabled
--time-zone STRING	time_zone
--use-default-time-zone	use_default_time_zone

Design notes

• Paired positive/negative boolean flags default to None so an unpassed flag never touches the server setting. A naive default=False would flip cache-warmup, subscriptions, etc. OFF on every editsite call.
• --time-zone and --use-default-time-zone are argparse-enforced mutually exclusive.
• Flags live in a new set_edit_site_only_args helper (not set_common_site_args) because create_site_command doesn't wire these attributes; adding to the shared helper would advertise unwired flags in createsite --help.
• --allow-mobile-snapshots maps onto sheet_image_enabled, NOT mobile_biometrics_enabled (a separate biometric-auth feature).
• --allow-subscriptions is inverted onto disable_subscriptions because that's how the REST API expresses the setting.

Upstream TSC bugs discovered (not fixed here)

RequestFactory.py lowercases both customSubscriptionEmail and customSubscriptionFooter on the wire, so user@Domain.com will be sent as user@domain.com. To be filed separately in server-client-python.

Test plan

• 30 new parser tests in tests/parsers/test_parser_edit_site.py cover each flag's arg form, mutual exclusion, and default=None behavior
• Full parsers suite: 127 passed, 1 skipped (pre-existing)

Closes #437.

🤖 Generated with Claude Code

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
tabcmd
__main__.py	12	12	12	0%
_version.py	11	11	11	0%
tabcmd.py	15	15	15	0%
version.py	9	5	5	44%
tabcmd/commands
commands.py	10	10	10	0%
constants.py	77	18	18	77%
server.py	135	18	18	87%
tabcmd/commands/auth
session.py	394	50	50	87%
tabcmd/commands/datasources_and_workbooks
datasources_and_workbooks_command.py	157	18	18	89%
datasources_workbooks_views_url_parser.py	142	5	5	96%
delete_command.py	60	16	16	73%
export_command.py	120	25	25	79%
get_url_command.py	127	47	47	63%
publish_command.py	123	28	28	77%
runschedule_command.py	21	7	7	67%
tabcmd/commands/extracts
create_extracts_command.py	42	8	8	81%
decrypt_extracts_command.py	27	2	2	93%
delete_extracts_command.py	37	6	6	84%
encrypt_extracts_command.py	27	2	2	93%
extracts.py	20	2	2	90%
reencrypt_extracts_command.py	27	2	2	93%
refresh_extracts_command.py	48	10	10	79%
tabcmd/commands/group
create_group_command.py	29	5	5	83%
delete_group_command.py	27	2	2	93%
tabcmd/commands/project
create_project_command.py	46	8	8	83%
delete_project_command.py	35	4	4	89%
publish_samples_command.py	30	4	4	87%
tabcmd/commands/site
create_site_command.py	34	5	5	85%
delete_site_command.py	27	2	2	93%
edit_site_command.py	65	17	17	74%
list_command.py	77	12	12	84%
list_sites_command.py	29	2	2	93%
tabcmd/commands/user
add_users_command.py	29	5	5	83%
create_site_users.py	58	11	11	81%
create_users_command.py	59	9	9	85%
delete_site_users_command.py	43	5	5	88%
user_data.py	222	31	31	86%
tabcmd/execution
_version.py	2	2	2	0%
global_options.py	146	8	8	95%
localize.py	66	11	11	83%
logger_config.py	60	6	6	90%
tabcmd_controller.py	42	7	7	83%
TOTAL	2921	473	84%