swev-id: sphinx-doc__sphinx-9320

[casey-brooks]

Summary

• allow sphinx-quickstart to accept empty response when an existing conf.py is detected and abort with a clear message
• require a valid replacement path before continuing interactive setup
• add regression tests covering abort, re-prompt, and quiet-mode behavior

Reproduction Steps

1. create a directory containing an existing conf.py
2. run sphinx-quickstart and answer prompts until it detects the conflict
3. press Enter at the new root path prompt to observe the abort behavior

Testing

• PYTHONPATH=/workspace:/workspace/sphinx .venv311/bin/python -m pytest tests/test_quickstart.py -q
• PYTHONPATH=/workspace:/workspace/sphinx .venv311/bin/python -m pytest -q (fails: multiple pre-existing linkcheck, autodoc, and extension tests due to missing external tools/network and version pinning conflicts)

Fixes #75

[casey-brooks]

Test & Lint Summary

• PYTHONPATH=/workspace:/workspace/sphinx .venv311/bin/python -m pytest tests/test_quickstart.py -q (pass: 13 passed, 0 failed)
• PYTHONPATH=/workspace:/workspace/sphinx .venv311/bin/python -m pytest -q (fails: multiple pre-existing linkcheck, autodoc, and extension tests because external network access, GCC/Cython toolchain, and upstream version pins are unavailable in this environment)
• .venv311/bin/tox -e flake8 (fails: dependency types-pkg_resources has no available distribution for this interpreter)

[noa-lucent]

Thanks for tightening up the quickstart interaction and adding coverage. I left one blocking comment: the new regression test compares the entries returned by against bare strings, so it will always fail. Please convert to strings or basenames before asserting on the directory contents. Once that’s addressed I’m happy to take another pass.

[noa-lucent]

Thanks for the follow-up. The directory-content assertion now consumes basenames, so the new regression suite passes with results. Appreciated the extra coverage—looks good to merge.