From e840a119dd8a2f68ba994ba7fdc099c504800384 Mon Sep 17 00:00:00 2001 From: malice Date: Sat, 23 May 2026 23:09:49 -0600 Subject: [PATCH] cleanup: drop dead deployment_config references + orphaned deleted_contacts template MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Tidies stale references left behind by the navi extraction + decoupling work. - lib/deployment_config.py: the consumer-catalog docstring listed four in-process consumers that were all extracted/removed across cleanups #4/#5/#6/#27 (/api/landclass gate, google_places.py, place_detail.py, offroute/router.py). Replaced the stale 4-bullet list with an accurate note: recon has no remaining caller of get_deployment_config() today; the module is retained per cleanup #1. - lib/api.py: removed the now-dead `from .deployment_config import get_deployment_config` import (its only caller was the /api/landclass handler removed in #5 — zero call sites remain). - templates/knowledge/deleted_contacts.html: deleted — orphaned since cleanup #3 removed the contacts/dashboard routes; zero callers in recon. No functional change (the removed import was unused; the template unrendered). Co-authored-by: Claude Opus 4.7 (1M context) --- lib/api.py | 1 - lib/deployment_config.py | 12 ++--- templates/knowledge/deleted_contacts.html | 56 ----------------------- 3 files changed, 6 insertions(+), 63 deletions(-) delete mode 100644 templates/knowledge/deleted_contacts.html diff --git a/lib/api.py b/lib/api.py index 576f4ee..a0697bf 100644 --- a/lib/api.py +++ b/lib/api.py @@ -24,7 +24,6 @@ from werkzeug.utils import secure_filename from .utils import get_config, content_hash, clean_filename_to_title, derive_source_and_category, generate_download_url, setup_logging from .status import StatusDB -from .deployment_config import get_deployment_config logger = setup_logging('recon.api') diff --git a/lib/deployment_config.py b/lib/deployment_config.py index 83cc864..ab6aa17 100644 --- a/lib/deployment_config.py +++ b/lib/deployment_config.py @@ -4,12 +4,12 @@ Deployment profile loader. Reads RECON_PROFILE env var (default: "home"), loads the matching YAML from config/profiles/.yaml, and caches the parsed dict in memory. -Provides get_deployment_config() for in-process consumers of the profile: - - lib/api.py:api_landclass — the has_landclass feature-flag gate - - lib/google_places.py — Google Places enrichment config - - lib/place_detail.py — place-detail enrichment config (×4 call sites) - - lib/offroute/router.py — profile.offroute.* (osm_pbf_path / postgis_dsn / - densify_interval_m) +Exposes get_deployment_config() as the in-process accessor for the profile. + +Note: its former consumers (the /api/landclass gate, google_places, +place_detail, offroute/router) were all extracted to navi-* services or removed +across cleanups #4–#6/#27 — recon has no remaining caller of +get_deployment_config() today; the module is retained per cleanup #1. (The former /api/config HTTP endpoint that served this dict to the frontend was removed once navi-config (:8422) took over that route.) """ diff --git a/templates/knowledge/deleted_contacts.html b/templates/knowledge/deleted_contacts.html deleted file mode 100644 index 58a9ff5..0000000 --- a/templates/knowledge/deleted_contacts.html +++ /dev/null @@ -1,56 +0,0 @@ -{% extends "base.html" %} -{% block content %} -

Deleted Contacts

-{% if not contacts %} -

No deleted contacts.

-{% else %} - - - {% for c in contacts %} - - - - - - - - - {% endfor %} -
LabelNameCategoryPhoneDeleted AtActions
{{ c.label }}{{ c.name or '' }}{{ c.category or '' }}{{ c.phone or '' }}{{ c.deleted_at or '' }} - - -
-{% endif %} -{% endblock %} -{% block scripts %} - -{% endblock %}