Fix duplicate get_dossier_output_dir() definition in config/settings.py

Root cause: an earlier delivery script was accidentally re-run after a
'git checkout -b' failed with branch-already-exists (bash does not stop
on a non-&&-chained command failure). That script's edit anchor matched
only the function's own unchanged body, not anything after it, so the
second run found the same anchor again and inserted a second, verbatim
copy of the function immediately after the first.

Trigger condition: any second invocation of that specific delivery step
against a repo where it had already succeeded once.

Blast radius: definition-order only. Python's later def silently
shadowed the earlier one; both bodies were byte-identical, so every
call site kept calling the same function with the same behavior. No
functional bug reached any user -- this is a code-hygiene / future-
drift-risk fix.

Verified (in sandbox before delivery): python -m py_compile succeeds;
AST-parsed the module and confirmed exactly one top-level FunctionDef
named get_dossier_output_dir (was 2, now 1); imported the module
directly and called get_dossier_output_dir() in both web mode and
BHARATGRAPH_DESKTOP_MODE=true, confirming both return the same paths
as before the fix.

config/settings.py

@@ -76,40 +76,6 @@ def get_dossier_output_dir() -> str:
     return os.path.join(_project_root, "data", "processed")
 
 
-def get_dossier_output_dir() -> str:
-    """Absolute base directory generated dossiers/reports are written to
-    and may be served from.
-
-    P-1 fix, centralized here because this exact path was previously
-    hardcoded/recomputed independently in THREE places that all need to
-    agree for the export flow to work at all:
-      - ai/dossier_generator.py's generate() computed its own
-        <project_root>/data as a path-traversal allowlist (H-07/CodeQL
-        fix) and joined output_dir onto <project_root> directly.
-      - api/routes/export.py passed the literal string "data/processed"
-        as that function's output_dir argument.
-      - api/routes/export.py ALSO independently recomputed
-        <project_root>/data/processed as ITS OWN separate, independent
-        path-traversal allowlist (a second H-07/CodeQL fix) to validate
-        the path generate() handed back, before serving the file.
-    These three were already subtly inconsistent with each other before
-    any desktop-mode change: the first allowed anything under
-    <project_root>/data/ (e.g. data/samples), while the second allowed
-    only <project_root>/data/processed specifically -- benign only
-    because output_dir is never actually varied by any real caller
-    today. Unifying all three onto this single function fixes that
-    latent inconsistency and means a future desktop-mode change to
-    where dossiers live can never desync between the two independent
-    security guards that gate the export endpoint.
-    Web-deployment behavior is unchanged when BHARATGRAPH_DESKTOP_MODE
-    is unset -- same exact <project_root>/data/processed as before.
-    """
-    if os.getenv("BHARATGRAPH_DESKTOP_MODE", "").lower() == "true":
-        return str(get_app_data_dir() / "reports")
-    _project_root = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
-    return os.path.join(_project_root, "data", "processed")
-
-
 def _local_config_path() -> Path:
     return get_app_data_dir() / "config.json"