diff --git a/apps/accounting/services/__init__.py b/apps/accounting/services/__init__.py
index bc430acd7..910d6dedc 100644
--- a/apps/accounting/services/__init__.py
+++ b/apps/accounting/services/__init__.py
@@ -6,11 +6,18 @@
 
     if apps.ready:
         from .core import JobAgingService, KPIService, StaffPerformanceService
+        from .finish_job_summary import (
+            FinishJobSummary,
+            build_finish_job_summary,
+            get_outstanding_invoiced_incl_tax,
+        )
         from .invoice_calculation import (
             InvoiceCalculationError,
             InvoiceCalculationResult,
+            JobInvoicingBasis,
             calculate_invoice_amount,
             get_job_for_invoice_calculation,
+            get_job_invoicing_basis,
             get_prior_valid_invoice_total,
         )
         from .payroll_reconciliation_service import PayrollReconciliationService
@@ -22,16 +29,21 @@
     pass
 
 __all__ = [
+    "FinishJobSummary",
     "InvoiceCalculationError",
     "InvoiceCalculationResult",
     "JobAgingService",
+    "JobInvoicingBasis",
     "KPIService",
     "PayrollReconciliationService",
     "RDTISpendService",
     "SalesPipelineService",
     "StaffPerformanceService",
     "WIPService",
+    "build_finish_job_summary",
     "calculate_invoice_amount",
     "get_job_for_invoice_calculation",
+    "get_job_invoicing_basis",
+    "get_outstanding_invoiced_incl_tax",
     "get_prior_valid_invoice_total",
 ]
diff --git a/apps/accounting/services/finish_job_summary.py b/apps/accounting/services/finish_job_summary.py
new file mode 100644
index 000000000..959f3948d
--- /dev/null
+++ b/apps/accounting/services/finish_job_summary.py
@@ -0,0 +1,80 @@
+"""Authoritative customer balance for the Finish Job workspace.
+
+Answers the counter question — what does this customer need to pay, including
+tax — as decimal currency values, so the frontend only formats what it is given
+(ADR 0020).
+
+Adds nothing but the tax and outstanding-balance arithmetic: the job's value, its
+price cap and the valid-invoice statuses all come from ``invoice_calculation``,
+so the amount a user can invoice and the amount they are shown cannot drift
+apart.
+"""
+
+from dataclasses import dataclass
+from decimal import ROUND_HALF_UP, Decimal
+
+from django.db.models import Sum
+from django.db.models.functions import Coalesce
+
+from apps.accounting.models.invoice import Invoice
+from apps.accounting.services.invoice_calculation import (
+    INVOICE_VALID_STATUSES,
+    get_job_invoicing_basis,
+    get_prior_valid_invoice_total,
+)
+from apps.job.models import Job
+from apps.workflow.models import CompanyDefaults
+
+CENT = Decimal("0.01")
+
+
+@dataclass(frozen=True)
+class FinishJobSummary:
+    """Every currency value the Finish Job workspace displays."""
+
+    job_value_excl_gst: Decimal
+    valid_invoiced_excl_gst: Decimal
+    outstanding_invoiced_incl_gst: Decimal
+    remaining_to_invoice_excl_gst: Decimal
+    remaining_gst: Decimal
+    remaining_to_invoice_incl_gst: Decimal
+    total_to_pay_incl_gst: Decimal
+    over_invoiced_excl_gst: Decimal
+
+
+def get_outstanding_invoiced_incl_tax(job: Job) -> Decimal:
+    """What Xero still expects to be paid on this job's valid invoices."""
+    return Decimal(
+        Invoice.objects.filter(
+            job_id=job.id, status__in=INVOICE_VALID_STATUSES
+        ).aggregate(total=Coalesce(Sum("amount_due"), Decimal("0")))["total"]
+    )
+
+
+def build_finish_job_summary(job: Job) -> FinishJobSummary:
+    job_value = get_job_invoicing_basis(job).target_total.quantize(
+        CENT, rounding=ROUND_HALF_UP
+    )
+    invoiced = get_prior_valid_invoice_total(job)
+    outstanding = get_outstanding_invoiced_incl_tax(job)
+
+    # A job invoiced beyond its value has nothing left to invoice; the excess is
+    # reported separately for resolution in Xero rather than netted off, which
+    # would hide it behind a smaller remaining balance.
+    remaining_excl = max(job_value - invoiced, Decimal("0"))
+    over_invoiced = max(invoiced - job_value, Decimal("0"))
+
+    gst_rate = CompanyDefaults.get_solo().gst_rate
+    remaining_gst = (remaining_excl * gst_rate).quantize(CENT, rounding=ROUND_HALF_UP)
+    remaining_incl = remaining_excl + remaining_gst
+
+    return FinishJobSummary(
+        job_value_excl_gst=job_value,
+        valid_invoiced_excl_gst=invoiced,
+        outstanding_invoiced_incl_gst=outstanding,
+        remaining_to_invoice_excl_gst=remaining_excl,
+        remaining_gst=remaining_gst,
+        remaining_to_invoice_incl_gst=remaining_incl,
+        total_to_pay_incl_gst=outstanding + remaining_incl,
+        over_invoiced_excl_gst=over_invoiced,
+    )
diff --git a/apps/accounting/services/invoice_calculation.py b/apps/accounting/services/invoice_calculation.py
index ac783abf0..44e9320a8 100644
--- a/apps/accounting/services/invoice_calculation.py
+++ b/apps/accounting/services/invoice_calculation.py
@@ -31,6 +31,38 @@ class InvoiceCalculationResult:
     calculated_amount: Decimal = Decimal("0")
 
 
+@dataclass(frozen=True)
+class JobInvoicingBasis:
+    """What a job is worth excluding tax, and which cost set says so."""
+
+    basis: str
+    target_total: Decimal
+
+
+def get_job_invoicing_basis(job: Job) -> JobInvoicingBasis:
+    """The complete value of a job excluding tax.
+
+    The single place a job's value is derived: fixed-price work is worth its
+    quote, T&M work its actual revenue limited by any price cap. Everything that
+    needs a job's value — invoice calculation, the Finish Job balance,
+    ``job_service.get_job_total_value`` — reads it from here, so the three cannot
+    disagree about what a job is worth.
+    """
+    if job.pricing_methodology == "fixed_price":
+        return JobInvoicingBasis(
+            basis="quote", target_total=Decimal(str(job.latest_quote.total_revenue))
+        )
+
+    actual_revenue = Decimal(str(job.latest_actual.total_revenue))
+    if job.price_cap is None:
+        return JobInvoicingBasis(basis="actual_revenue", target_total=actual_revenue)
+
+    return JobInvoicingBasis(
+        basis="actual_revenue",
+        target_total=min(actual_revenue, Decimal(str(job.price_cap))),
+    )
+
+
 def get_prior_valid_invoice_total(job: Job) -> Decimal:
     return Decimal(
         Invoice.objects.filter(
@@ -40,9 +72,13 @@ def get_prior_valid_invoice_total(job: Job) -> Decimal:
 
 
 def get_job_for_invoice_calculation(job_id: UUID) -> Job:
-    job = Job.objects.select_related("company", "latest_quote", "latest_actual").get(
-        id=job_id
-    )
+    """Load a job with the cost set its value is measured against.
+
+    Only the basis cost set is fetched: loading both eagerly pulls one that is
+    never read, which the n+1 guard reports. Its cost lines are prefetched
+    because ``total_revenue`` sums them.
+    """
+    job = Job.objects.select_related("company").get(id=job_id)
 
     if job.pricing_methodology == "fixed_price":
         prefetch_related_objects([job], "latest_quote__cost_lines")
@@ -59,27 +95,26 @@ def calculate_invoice_amount(
     amount: Decimal | None = None,
 ) -> InvoiceCalculationResult:
     prior_invoiced = get_prior_valid_invoice_total(job)
+    job_basis = get_job_invoicing_basis(job)
 
+    # Which modes are legal is a different question from what the job is worth,
+    # so it keeps its own branch; the handlers below are given the value.
     if job.pricing_methodology == "fixed_price":
-        return _calculate_fixed_price(job, mode, prior_invoiced, percent, amount)
+        return _calculate_fixed_price(job_basis, mode, prior_invoiced, percent, amount)
     else:
-        return _calculate_time_materials(job, mode, prior_invoiced, percent, amount)
+        return _calculate_time_materials(
+            job_basis, mode, prior_invoiced, percent, amount
+        )
 
 
 def _calculate_fixed_price(
-    job: Job,
+    job_basis: JobInvoicingBasis,
     mode: str,
     prior_invoiced: Decimal,
     percent: Decimal | None,
     amount: Decimal | None,
 ) -> InvoiceCalculationResult:
-    quote = job.latest_quote
-    if not quote:
-        raise InvoiceCalculationError(
-            "Fixed-price job has no quote to invoice against."
-        )
-    target_total = Decimal(str(quote.total_revenue))
-    target_basis = "quote"
+    target_total = job_basis.target_total
 
     if mode == "invoice_full":
         calculated = target_total - prior_invoiced
@@ -108,7 +143,7 @@ def _calculate_fixed_price(
 
     return InvoiceCalculationResult(
         mode=mode,
-        target_basis=target_basis,
+        target_basis=job_basis.basis,
         target_total=target_total,
         prior_invoiced_total=prior_invoiced,
         requested_percent=percent,
@@ -118,23 +153,13 @@ def _calculate_fixed_price(
 
 
 def _calculate_time_materials(
-    job: Job,
+    job_basis: JobInvoicingBasis,
     mode: str,
     prior_invoiced: Decimal,
     percent: Decimal | None,
     amount: Decimal | None,
 ) -> InvoiceCalculationResult:
-    actual = job.latest_actual
-    if not actual:
-        raise InvoiceCalculationError(
-            "T&M job has no actual cost set to invoice against."
-        )
-    target_total = Decimal(str(actual.total_revenue))
-
-    if job.price_cap is not None:
-        target_total = min(target_total, Decimal(str(job.price_cap)))
-
-    target_basis = "actual_revenue"
+    target_total = job_basis.target_total
 
     if mode == "invoice_costs_to_date":
         calculated = target_total - prior_invoiced
@@ -158,7 +183,7 @@ def _calculate_time_materials(
 
     return InvoiceCalculationResult(
         mode=mode,
-        target_basis=target_basis,
+        target_basis=job_basis.basis,
         target_total=target_total,
         prior_invoiced_total=prior_invoiced,
         requested_percent=percent,
diff --git a/apps/accounting/tests/test_finish_job_summary.py b/apps/accounting/tests/test_finish_job_summary.py
new file mode 100644
index 000000000..25667f442
--- /dev/null
+++ b/apps/accounting/tests/test_finish_job_summary.py
@@ -0,0 +1,234 @@
+"""Tests for the Finish Job customer balance.
+
+The counter situations from KAN-323 are all the same arithmetic over different
+invoice states, so they are one table. The cases that are not just arithmetic —
+the price cap, excluded statuses, over-invoicing and GST rounding — get their own
+tests.
+"""
+
+import uuid
+from datetime import date
+from decimal import Decimal
+
+from django.utils import timezone
+
+from apps.accounting.models.invoice import Invoice
+from apps.accounting.services.finish_job_summary import build_finish_job_summary
+from apps.company.models import Company
+from apps.job.models import Job
+from apps.job.models.costing import CostSet
+from apps.testing import BaseTestCase
+from apps.workflow.models import CompanyDefaults
+
+
+class TestFinishJobSummary(BaseTestCase):
+    def setUp(self) -> None:
+        self.client_obj = Company.objects.create(
+            name="Test Company",
+            xero_last_modified=timezone.now(),
+        )
+        defaults = CompanyDefaults.get_solo()
+        defaults.gst_rate = Decimal("0.1500")
+        defaults.save()
+
+    def _job(self, pricing_methodology: str, revenue: Decimal) -> Job:
+        job = Job(
+            company=self.client_obj,
+            name="Test Job",
+            pricing_methodology=pricing_methodology,
+        )
+        job.save(staff=self.test_staff)
+        basis = (
+            job.latest_quote
+            if pricing_methodology == "fixed_price"
+            else job.latest_actual
+        )
+        self._add_revenue_line(basis, revenue)
+        return job
+
+    def _add_revenue_line(self, cost_set: CostSet, revenue: Decimal) -> None:
+        from apps.job.models.costing import CostLine
+
+        CostLine.objects.create(
+            cost_set=cost_set,
+            kind="adjust",
+            desc="Test line",
+            quantity=Decimal("1.000"),
+            unit_cost=Decimal("0.00"),
+            unit_rev=revenue,
+            accounting_date=date.today(),
+        )
+
+    def _invoice(
+        self,
+        job: Job,
+        amount: Decimal,
+        amount_due: Decimal = Decimal("0.00"),
+        status: str = "AUTHORISED",
+    ) -> Invoice:
+        return Invoice.objects.create(
+            job=job,
+            company=self.client_obj,
+            xero_id=uuid.uuid4(),
+            number=f"INV-{uuid.uuid4().hex[:8]}",
+            status=status,
+            total_excl_tax=amount,
+            tax=(amount * Decimal("0.15")).quantize(Decimal("0.01")),
+            total_incl_tax=amount * Decimal("1.15"),
+            amount_due=amount_due,
+            date=date.today(),
+            xero_last_modified=timezone.now(),
+            raw_json={},
+        )
+
+    def test_counter_situations(self) -> None:
+        """Every KAN-323 acceptance scenario, on a $1000 fixed-price job."""
+        cases = [
+            # label, invoices as (excl_tax, amount_due, status), expected total to pay
+            ("nothing invoiced", [], Decimal("1150.00")),
+            (
+                "paid in advance",
+                [(Decimal("1000"), Decimal("0.00"), "PAID")],
+                Decimal("0.00"),
+            ),
+            (
+                "unpaid advance invoice",
+                [(Decimal("1000"), Decimal("1150.00"), "AUTHORISED")],
+                Decimal("1150.00"),
+            ),
+            (
+                "paid deposit, balance uninvoiced",
+                [(Decimal("400"), Decimal("0.00"), "PAID")],
+                Decimal("690.00"),
+            ),
+            (
+                "unpaid progress invoice plus uninvoiced balance",
+                [(Decimal("400"), Decimal("460.00"), "AUTHORISED")],
+                Decimal("1150.00"),
+            ),
+            (
+                "partly paid invoice",
+                [(Decimal("1000"), Decimal("150.00"), "AUTHORISED")],
+                Decimal("150.00"),
+            ),
+        ]
+
+        for label, invoices, expected in cases:
+            with self.subTest(label):
+                job = self._job("fixed_price", Decimal("1000"))
+                for amount, due, status in invoices:
+                    self._invoice(job, amount, amount_due=due, status=status)
+
+                summary = build_finish_job_summary(job)
+
+                self.assertEqual(summary.total_to_pay_incl_gst, expected)
+
+    def test_time_materials_value_is_limited_by_price_cap(self) -> None:
+        job = self._job("time_materials", Decimal("5000"))
+        job.price_cap = Decimal("3000.00")
+        job.save(staff=self.test_staff)
+
+        summary = build_finish_job_summary(job)
+
+        self.assertEqual(summary.job_value_excl_gst, Decimal("3000.00"))
+        self.assertEqual(summary.remaining_to_invoice_excl_gst, Decimal("3000.00"))
+
+    def test_voided_and_deleted_invoices_do_not_count(self) -> None:
+        job = self._job("fixed_price", Decimal("1000"))
+        self._invoice(job, Decimal("1000"), Decimal("1150.00"), "VOIDED")
+        self._invoice(job, Decimal("1000"), Decimal("1150.00"), "DELETED")
+
+        summary = build_finish_job_summary(job)
+
+        self.assertEqual(summary.valid_invoiced_excl_gst, Decimal("0.00"))
+        self.assertEqual(summary.outstanding_invoiced_incl_gst, Decimal("0.00"))
+        self.assertEqual(summary.total_to_pay_incl_gst, Decimal("1150.00"))
+
+    def test_over_invoicing_is_reported_without_a_negative_remainder(self) -> None:
+        """T&M actuals can land below what was already invoiced."""
+        job = self._job("time_materials", Decimal("800"))
+        self._invoice(job, Decimal("1000"), Decimal("0.00"), "PAID")
+
+        summary = build_finish_job_summary(job)
+
+        self.assertEqual(summary.over_invoiced_excl_gst, Decimal("200.00"))
+        self.assertEqual(summary.remaining_to_invoice_excl_gst, Decimal("0.00"))
+        self.assertEqual(summary.total_to_pay_incl_gst, Decimal("0.00"))
+
+    def test_gst_rounds_to_the_nearest_cent(self) -> None:
+        """$333.33 x 15% = $49.9995, which must present as $50.00."""
+        job = self._job("time_materials", Decimal("333.33"))
+
+        summary = build_finish_job_summary(job)
+
+        self.assertEqual(summary.remaining_gst, Decimal("50.00"))
+        self.assertEqual(summary.total_to_pay_incl_gst, Decimal("383.33"))
+
+    def test_gst_uses_the_configured_rate(self) -> None:
+        defaults = CompanyDefaults.get_solo()
+        defaults.gst_rate = Decimal("0.1250")
+        defaults.save()
+        job = self._job("time_materials", Decimal("1000"))
+
+        summary = build_finish_job_summary(job)
+
+        self.assertEqual(summary.remaining_gst, Decimal("125.00"))
+        self.assertEqual(summary.total_to_pay_incl_gst, Decimal("1125.00"))
+
+
+class TestJobValueIsSharedWithReporting(BaseTestCase):
+    """get_job_total_value must agree with the invoicing basis.
+
+    It used to read CostSet.summary["rev"], a float mirror maintained by
+    CostLine._update_cost_set_summary, while invoicing read total_revenue. Both
+    now come from get_job_invoicing_basis, so a job cannot be reported at one
+    value and invoiced at another.
+    """
+
+    def setUp(self) -> None:
+        self.client_obj = Company.objects.create(
+            name="Test Company",
+            xero_last_modified=timezone.now(),
+        )
+
+    def _job_with_revenue(self, revenue: Decimal) -> Job:
+        from apps.job.models.costing import CostLine
+
+        job = Job(
+            company=self.client_obj,
+            name="Test Job",
+            pricing_methodology="time_materials",
+        )
+        job.save(staff=self.test_staff)
+        CostLine.objects.create(
+            cost_set=job.latest_actual,
+            kind="adjust",
+            desc="Test line",
+            quantity=Decimal("1.000"),
+            unit_cost=Decimal("0.00"),
+            unit_rev=revenue,
+            accounting_date=date.today(),
+        )
+        return job
+
+    def test_reported_value_matches_the_invoicing_basis(self) -> None:
+        from apps.accounting.services.invoice_calculation import (
+            get_job_invoicing_basis,
+        )
+        from apps.job.services.job_service import get_job_total_value
+
+        job = self._job_with_revenue(Decimal("1234.56"))
+
+        self.assertEqual(
+            get_job_total_value(job), get_job_invoicing_basis(job).target_total
+        )
+
+    def test_reported_value_respects_the_price_cap(self) -> None:
+        """The old implementation ignored price_cap and over-reported."""
+        from apps.job.services.job_service import get_job_total_value
+
+        job = self._job_with_revenue(Decimal("5000"))
+        job.price_cap = Decimal("3000.00")
+        job.save(staff=self.test_staff)
+
+        self.assertEqual(get_job_total_value(job), Decimal("3000.00"))
diff --git a/apps/accounts/tests/test_auth_observability.py b/apps/accounts/tests/test_auth_observability.py
index d82cef775..f08932e74 100644
--- a/apps/accounts/tests/test_auth_observability.py
+++ b/apps/accounts/tests/test_auth_observability.py
@@ -26,15 +26,16 @@ def test_jwt_auth_logs_cookie_miss_with_request_context() -> None:
     with patch.object(authentication.logger, "info") as log_info:
         assert JWTAuthentication().authenticate(request) is None
 
-    log_info.assert_called_once_with(
-        "JWT AUTH MISS - method=%s path=%s ip=%s access_cookie=%s access_cookie_present=%s refresh_cookie_present=%s",
+    log_info.assert_called_once()
+    _, *log_args = log_info.call_args.args
+    assert log_args == [
         "GET",
         "/api/accounts/me/",
         TEST_CLIENT_IP,
         "access_token",
         False,
         False,
-    )
+    ]
 
 
 @override_settings(ENABLE_JWT_AUTH=True)
@@ -111,12 +112,9 @@ def test_logout_logs_cookie_presence_without_token_values() -> None:
         response = LogoutUserAPIView.as_view()(request)
 
     assert response.status_code == 200
-    log_info.assert_called_once_with(
-        "JWT LOGOUT REQUEST - ip=%s access_cookie_present=%s refresh_cookie_present=%s",
-        TEST_CLIENT_IP,
-        True,
-        True,
-    )
+    log_info.assert_called_once()
+    _, *log_args = log_info.call_args.args
+    assert log_args == [TEST_CLIENT_IP, True, True]
 
 
 @pytest.mark.django_db
diff --git a/apps/job/migrations/0010_job_completion_checklist_fields.py b/apps/job/migrations/0010_job_completion_checklist_fields.py
new file mode 100644
index 000000000..99b380680
--- /dev/null
+++ b/apps/job/migrations/0010_job_completion_checklist_fields.py
@@ -0,0 +1,53 @@
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    """Front-desk completion checklist as Job fields.
+
+    ``collected`` is renamed rather than replaced: it meant the same thing and was
+    never writable (no serializer, endpoint, admin or UI ever set it), so every
+    row holds the default and the rename carries no stale data forward.
+    """
+
+    dependencies = [
+        ("job", "0009_text_unset_constraints"),
+    ]
+
+    operations = [
+        migrations.RenameField(
+            model_name="job",
+            old_name="collected",
+            new_name="released",
+        ),
+        migrations.AlterField(
+            model_name="job",
+            name="released",
+            field=models.BooleanField(
+                default=False,
+                help_text=(
+                    "The job has left our possession, by collection, delivery or "
+                    "on-site install."
+                ),
+            ),
+        ),
+        migrations.AddField(
+            model_name="job",
+            name="foreman_signed_off",
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name="job",
+            name="timesheets_collected",
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name="job",
+            name="materials_checked",
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name="job",
+            name="customer_called",
+            field=models.BooleanField(default=False),
+        ),
+    ]
diff --git a/apps/job/models/job.py b/apps/job/models/job.py
index 66fed6ab1..aff619284 100644
--- a/apps/job/models/job.py
+++ b/apps/job/models/job.py
@@ -154,6 +154,17 @@ class Job(models.Model):
         "is_urgent",
     ]
 
+    # The front-desk completion checklist, in display order. The Finish Job
+    # serializers derive their field lists from this, so an item is declared
+    # once.
+    COMPLETION_CHECKLIST_FIELDS = [
+        "foreman_signed_off",
+        "timesheets_collected",
+        "materials_checked",
+        "customer_called",
+        "released",
+    ]
+
     # Fields where changes are NOT audited via JobEvent. Every field NOT in
     # this set is automatically tracked — adding a new business field to Job
     # gives you audit coverage for free.
@@ -307,8 +318,19 @@ class Job(models.Model):
     # Shop job has no company (company_id is None)
 
     job_is_valid = models.BooleanField(default=False)
-    collected: bool = models.BooleanField(default=False)
     paid: bool = models.BooleanField(default=False)
+
+    # Front-desk completion checklist. Each is a fact only a person knows, so
+    # none of them can be derived. Ticking one writes a job-history event through
+    # _FIELD_HANDLERS below; nothing here blocks invoicing or moves the job.
+    foreman_signed_off = models.BooleanField(default=False)
+    timesheets_collected = models.BooleanField(default=False)
+    materials_checked = models.BooleanField(default=False)
+    customer_called = models.BooleanField(default=False)
+    released = models.BooleanField(
+        default=False,
+        help_text="The job has left our possession, by collection, delivery or on-site install.",
+    )
     fully_invoiced: bool = models.BooleanField(
         default=False,
         help_text="The total value of invoices for this job matches the total value of the job.",
@@ -1146,10 +1168,30 @@ def _handle_rdti_type_change(_self_job, old_type, new_type):
             "payment_updated",
             "Paid",
         ),
-        "collected": _handle_boolean_change(
-            "job_collected",
-            "collection_updated",
-            "Collected",
+        "foreman_signed_off": _handle_boolean_change(
+            "completion_checklist_updated",
+            "completion_checklist_updated",
+            "Foreman sign-off",
+        ),
+        "timesheets_collected": _handle_boolean_change(
+            "completion_checklist_updated",
+            "completion_checklist_updated",
+            "Timesheets collected",
+        ),
+        "materials_checked": _handle_boolean_change(
+            "completion_checklist_updated",
+            "completion_checklist_updated",
+            "Materials checked",
+        ),
+        "customer_called": _handle_boolean_change(
+            "completion_checklist_updated",
+            "completion_checklist_updated",
+            "Customer called",
+        ),
+        "released": _handle_boolean_change(
+            "completion_checklist_updated",
+            "completion_checklist_updated",
+            "Job released",
         ),
         "complex_job": _handle_boolean_change(
             "job_updated",
diff --git a/apps/job/models/job_event.py b/apps/job/models/job_event.py
index 9ed242b77..32fa55e00 100644
--- a/apps/job/models/job_event.py
+++ b/apps/job/models/job_event.py
@@ -1,5 +1,6 @@
 import hashlib
 import uuid
+from collections.abc import Callable
 from datetime import timedelta
 
 from django.core.exceptions import ValidationError
@@ -47,6 +48,25 @@ def _truncate_change(label: str, old, new) -> str:
     return f"{label} changed from '{_truncate(old)}' to '{_truncate(new)}'"
 
 
+def _completion_confirmation_descriptor(
+    confirmed: str, withdrawn: str
+) -> Callable[[str, str], str]:
+    """Descriptor factory for front-desk checklist items.
+
+    Both wordings are given explicitly rather than built from one subject: an
+    unticking is the change most worth finding later, so it earns a sentence that
+    reads properly instead of a negated one that does not.
+    """
+
+    def descriptor(old: str, new: str) -> str:
+        if _truthy(new):
+            return confirmed
+        else:
+            return withdrawn
+
+    return descriptor
+
+
 def _quote_acceptance_descriptor(old, new) -> str:
     if new and not old:
         return f"Quote accepted on {new}"
@@ -65,10 +85,22 @@ def _quote_acceptance_descriptor(old, new) -> str:
         "Marked as complex job" if _truthy(new) else "Unmarked as complex job"
     ),
     "Paid": lambda old, new: ("Marked as paid" if _truthy(new) else "Marked as unpaid"),
-    "Collected": lambda old, new: (
-        "Marked as collected" if _truthy(new) else "Marked as not collected"
-    ),
     "Quote acceptance date": _quote_acceptance_descriptor,
+    "Foreman sign-off": _completion_confirmation_descriptor(
+        "Foreman signed the job off", "Foreman sign-off withdrawn"
+    ),
+    "Timesheets collected": _completion_confirmation_descriptor(
+        "Timesheets collected from the workshop", "Timesheet collection unconfirmed"
+    ),
+    "Materials checked": _completion_confirmation_descriptor(
+        "Materials checked on the job", "Materials check unconfirmed"
+    ),
+    "Customer called": _completion_confirmation_descriptor(
+        "Customer called", "Customer call unconfirmed"
+    ),
+    "Job released": _completion_confirmation_descriptor(
+        "Job released", "Job release withdrawn"
+    ),
     "Internal notes": lambda old, new: _truncate_change("Notes", old, new),
     "Job description": lambda old, new: _truncate_change("Description", old, new),
     "Notes": lambda old, new: _truncate_change("Notes", old, new),
@@ -324,28 +356,29 @@ def _build_jsa_description(detail: dict) -> str:
         return f"JSA generated: {title}"
 
     _DESCRIPTION_BUILDERS = {
-        "job_created": _build_job_created_description.__func__,
-        "status_changed": _build_status_changed_description.__func__,
-        "job_updated": _build_changes_description.__func__,
-        "company_changed": _build_changes_description.__func__,
-        "person_changed": _build_changes_description.__func__,
-        "notes_updated": _build_changes_description.__func__,
-        "delivery_date_changed": _build_changes_description.__func__,
-        "quote_accepted": _build_changes_description.__func__,
-        "pricing_changed": _build_changes_description.__func__,
-        "priority_changed": _build_priority_changed_description.__func__,
-        "payment_received": _build_changes_description.__func__,
-        "payment_updated": _build_changes_description.__func__,
-        "job_collected": _build_changes_description.__func__,
-        "collection_updated": _build_changes_description.__func__,
-        "job_rejected": _build_changes_description.__func__,
-        "manual_note": _build_manual_note_description.__func__,
-        "invoice_created": _build_invoice_created_description.__func__,
-        "invoice_deleted": _build_invoice_deleted_description.__func__,
-        "quote_created": _build_quote_created_description.__func__,
-        "quote_deleted": _build_quote_deleted_description.__func__,
-        "delivery_docket_generated": _build_delivery_docket_description.__func__,
-        "jsa_generated": _build_jsa_description.__func__,
+        "job_created": _build_job_created_description,
+        "status_changed": _build_status_changed_description,
+        "job_updated": _build_changes_description,
+        "company_changed": _build_changes_description,
+        "person_changed": _build_changes_description,
+        "notes_updated": _build_changes_description,
+        "delivery_date_changed": _build_changes_description,
+        "quote_accepted": _build_changes_description,
+        "pricing_changed": _build_changes_description,
+        "priority_changed": _build_priority_changed_description,
+        "payment_received": _build_changes_description,
+        "payment_updated": _build_changes_description,
+        "job_collected": _build_changes_description,
+        "collection_updated": _build_changes_description,
+        "job_rejected": _build_changes_description,
+        "completion_checklist_updated": _build_changes_description,
+        "manual_note": _build_manual_note_description,
+        "invoice_created": _build_invoice_created_description,
+        "invoice_deleted": _build_invoice_deleted_description,
+        "quote_created": _build_quote_created_description,
+        "quote_deleted": _build_quote_deleted_description,
+        "delivery_docket_generated": _build_delivery_docket_description,
+        "jsa_generated": _build_jsa_description,
     }
 
     class Meta:
diff --git a/apps/job/serializers/__init__.py b/apps/job/serializers/__init__.py
index 61be51055..f8c52bb8f 100644
--- a/apps/job/serializers/__init__.py
+++ b/apps/job/serializers/__init__.py
@@ -82,12 +82,16 @@
             AssignJobSerializer,
             CompanyDefaultsJobDetailSerializer,
             CompleteJobSerializer,
+            FinishJobPayload,
+            FinishJobSummarySerializer,
             GroupedJobDeltaRejectionListResponseSerializer,
             GroupedJobDeltaRejectionResolveRequestSerializer,
             GroupedJobDeltaRejectionResolveResponseSerializer,
             GroupedJobDeltaRejectionSerializer,
             InvoiceSerializer,
             JobBasicInformationResponseSerializer,
+            JobCompletionChecklistSerializer,
+            JobCompletionChecklistUpdateSerializer,
             JobCostSetSummarySerializer,
             JobCostSummaryResponseSerializer,
             JobCreateResponseSerializer,
@@ -102,6 +106,7 @@
             JobEventCreateSerializer,
             JobEventSerializer,
             JobEventsResponseSerializer,
+            JobFinishResponseSerializer,
             JobHeaderResponseSerializer,
             JobInvoicesResponseSerializer,
             JobPatchSerializer,
@@ -224,12 +229,16 @@
     "FetchJobsResponseSerializer",
     "FetchStatusValuesResponseSerializer",
     "FiltersAppliedSerializer",
+    "FinishJobPayload",
+    "FinishJobSummarySerializer",
     "GroupedJobDeltaRejectionListResponseSerializer",
     "GroupedJobDeltaRejectionResolveRequestSerializer",
     "GroupedJobDeltaRejectionResolveResponseSerializer",
     "GroupedJobDeltaRejectionSerializer",
     "InvoiceSerializer",
     "JobBasicInformationResponseSerializer",
+    "JobCompletionChecklistSerializer",
+    "JobCompletionChecklistUpdateSerializer",
     "JobCostSetSummarySerializer",
     "JobCostSummaryResponseSerializer",
     "JobCreateResponseSerializer",
@@ -252,6 +261,7 @@
     "JobFileUploadSerializer",
     "JobFileUploadSuccessResponseSerializer",
     "JobFileUploadViewResponseSerializer",
+    "JobFinishResponseSerializer",
     "JobHeaderResponseSerializer",
     "JobInvoicesResponseSerializer",
     "JobLabourRateSerializer",
diff --git a/apps/job/serializers/job_serializer.py b/apps/job/serializers/job_serializer.py
index 94aa9bb59..e6ef86c9b 100644
--- a/apps/job/serializers/job_serializer.py
+++ b/apps/job/serializers/job_serializer.py
@@ -1,6 +1,6 @@
 import json
 import logging
-from typing import Any, Optional
+from typing import Any, Optional, TypedDict
 
 from django.core.exceptions import ObjectDoesNotExist
 from drf_spectacular.utils import extend_schema_field
@@ -8,6 +8,7 @@
 
 from apps.accounting.models.invoice import Invoice
 from apps.accounting.models.quote import Quote
+from apps.accounting.services.finish_job_summary import FinishJobSummary
 from apps.company.models import Company, Person
 from apps.job.models import Job, JobEvent, JobFile
 from apps.workflow.models import XeroPayItem
@@ -847,6 +848,88 @@ class JobCostSummaryResponseSerializer(serializers.Serializer):
     actual = JobCostSetSummarySerializer(allow_null=True)
 
 
+class FinishJobPayload(TypedDict):
+    """What the Finish Job endpoint serializes: a balance plus a checklist."""
+
+    summary: FinishJobSummary
+    checklist: Job
+
+
+class FinishJobSummarySerializer(serializers.Serializer[FinishJobSummary]):
+    """The authoritative customer balance shown in the Finish Job workspace.
+
+    Read-only: every value is calculated by
+    apps.accounting.services.finish_job_summary, and the frontend formats rather
+    than recomputes them (ADR 0020).
+    """
+
+    job_value_excl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    valid_invoiced_excl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    outstanding_invoiced_incl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    remaining_to_invoice_excl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    remaining_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    remaining_to_invoice_incl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    total_to_pay_incl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+    over_invoiced_excl_gst = serializers.DecimalField(
+        max_digits=12, decimal_places=2, read_only=True
+    )
+
+
+class JobCompletionChecklistSerializer(NullUnsetModelSerializer[Job]):
+    """Read shape for the front-desk completion checklist.
+
+    The items are Job fields, so each tick is audited by the job's own
+    field-change machinery. Who ticked what, and when, is in the job history.
+    """
+
+    class Meta:
+        model = Job
+        fields = Job.COMPLETION_CHECKLIST_FIELDS
+        read_only_fields = fields
+
+
+class JobCompletionChecklistUpdateSerializer(NullUnsetModelSerializer[Job]):
+    """Partial update shape: send only the items being changed.
+
+    Unknown keys are rejected rather than dropped, so a client typo is a 400
+    instead of a silent no-op.
+    """
+
+    class Meta:
+        model = Job
+        fields = Job.COMPLETION_CHECKLIST_FIELDS
+        extra_kwargs = {field: {"required": False} for field in fields}
+
+    def validate(self, attrs: dict[str, bool]) -> dict[str, bool]:
+        unknown = sorted(set(self.initial_data) - set(self.fields))
+        if unknown:
+            raise serializers.ValidationError(
+                f"Unknown checklist item(s): {', '.join(unknown)}."
+            )
+        return attrs
+
+
+class JobFinishResponseSerializer(serializers.Serializer[FinishJobPayload]):
+    """Everything the Finish Job workspace reads in one request."""
+
+    summary = FinishJobSummarySerializer(read_only=True)
+    checklist = JobCompletionChecklistSerializer(read_only=True)
+
+
 class JobEventsResponseSerializer(serializers.Serializer):
     """Serializer for job events response"""
 
diff --git a/apps/job/services/__init__.py b/apps/job/services/__init__.py
index 6eb43df58..95517dbb3 100644
--- a/apps/job/services/__init__.py
+++ b/apps/job/services/__init__.py
@@ -43,6 +43,7 @@
             get_job_total_value,
             get_paid_complete_jobs,
             recalculate_job_invoicing_state,
+            update_completion_checklist,
         )
         from .job_summary_pdf_service import JobSummaryPdfService
         from .kanban_categorization_service import (
@@ -185,5 +186,6 @@
     "serialize_validation_report",
     "sync_job_folder",
     "to_decimal",
+    "update_completion_checklist",
     "wait_until_file_ready",
 ]
diff --git a/apps/job/services/job_service.py b/apps/job/services/job_service.py
index 308ddba86..f472cbe5a 100644
--- a/apps/job/services/job_service.py
+++ b/apps/job/services/job_service.py
@@ -2,12 +2,14 @@
 from decimal import Decimal
 
 from django.db import transaction
-from django.db.models import Sum
-from django.db.models.functions import Coalesce
 from django.utils import timezone
 
-from apps.accounting.enums import InvoiceStatus
 from apps.accounting.models.invoice import Invoice
+from apps.accounting.services.invoice_calculation import (
+    INVOICE_VALID_STATUSES,
+    get_job_invoicing_basis,
+    get_prior_valid_invoice_total,
+)
 from apps.accounts.models import Staff
 from apps.job.models import Job
 from apps.workflow.services.error_persistence import persist_app_error
@@ -47,56 +49,39 @@ def archive_complete_jobs(job_ids, staff=None):
 
 
 def get_job_total_value(job: Job) -> Decimal:
-    """
-    Get the total value of a job using the definitive logic:
-    1. If invoiced: Use total invoice amount (definitive)
-    2. Else if quote job: Use quote revenue
-    3. Else (T&M): Use actual revenue
-
-    Args:
-        job: Job instance
+    """The total value of a job.
 
-    Returns:
-        Decimal: Total job value
+    Invoices are definitive once raised; before that the job is worth whatever
+    its pricing basis says. Both figures come from ``invoice_calculation``, so
+    this cannot report a value the job could not actually be invoiced for.
     """
-    # Check for invoices first - they override everything
-    INVOICE_VALID_STATUSES = [
-        status
-        for (status, _) in InvoiceStatus.choices
-        if status not in ["VOIDED", "DELETED"]
-    ]
-
-    total_invoiced = Decimal(
-        Invoice.objects.filter(
-            job_id=job.id, status__in=INVOICE_VALID_STATUSES
-        ).aggregate(total=Coalesce(Sum("total_excl_tax"), Decimal("0")))["total"]
-    )
-
+    total_invoiced = get_prior_valid_invoice_total(job)
     if total_invoiced > 0:
         return total_invoiced
 
-    # No invoices - check pricing methodology
-    if job.pricing_methodology == "fixed_price":
-        quote = job.get_latest("quote")
-        if quote and quote.summary:
-            return Decimal(str(quote.summary.get("rev", 0)))
-        return Decimal("0.00")
-    else:
-        # T&M job - use actual
-        actual = job.get_latest("actual")
-        if actual and actual.summary:
-            return Decimal(str(actual.summary.get("rev", 0)))
-        return Decimal("0.00")
+    return get_job_invoicing_basis(job).target_total
+
+
+def update_completion_checklist(
+    job: Job, updates: dict[str, bool], staff: Staff
+) -> Job:
+    """Tick or untick front-desk checklist items.
+
+    Job.save() turns each changed field into a job-history event through
+    _FIELD_HANDLERS, so there is no audit code here to keep in step. Validation
+    of the keys and values belongs to the serializer that accepted them.
+    """
+    for item, value in updates.items():
+        setattr(job, item, value)
+    # Scoped to the ticked items so a concurrent write to an unrelated field —
+    # fully_invoiced, status, costing — is not overwritten with the value this
+    # request read before the user touched the checkbox.
+    job.save(staff=staff, update_fields=list(updates.keys()))
+    return job
 
 
 def recalculate_job_invoicing_state(job_id: str, staff) -> None:
     try:
-        INVOICE_VALID_STATUSES = [
-            status
-            for (status, _) in InvoiceStatus.choices
-            if status not in ["VOIDED", "DELETED"]
-        ]
-
         has_invoices = Invoice.objects.filter(
             job_id=job_id, status__in=INVOICE_VALID_STATUSES
         ).exists()
@@ -111,19 +96,13 @@ def recalculate_job_invoicing_state(job_id: str, staff) -> None:
 
         job = Job.objects.select_related("latest_actual", "latest_quote").get(pk=job_id)
 
-        total_invoiced = Decimal(
-            Invoice.objects.filter(
-                job_id=job_id, status__in=INVOICE_VALID_STATUSES
-            ).aggregate(total=Coalesce(Sum("total_excl_tax"), Decimal("0")))["total"]
+        # Compared against the same value the job would be invoiced for, so a
+        # capped T&M job cannot read "fully invoiced" at one figure while the
+        # invoice path uses another.
+        job.fully_invoiced = (
+            get_prior_valid_invoice_total(job)
+            >= get_job_invoicing_basis(job).target_total
         )
-
-        # Fixed-price jobs compare against quote; T&M against actuals
-        if job.pricing_methodology == "fixed_price" and job.latest_quote:
-            target_amount = Decimal(str(job.latest_quote.total_revenue))
-        else:
-            target_amount = Decimal(str(job.latest_actual.total_revenue))
-
-        job.fully_invoiced = total_invoiced >= target_amount
         job.save(staff=staff, update_fields=["fully_invoiced", "updated_at"])
     except Job.DoesNotExist:
         logger.error("Provided job id doesn't exist")
diff --git a/apps/job/tests/test_completion_checklist.py b/apps/job/tests/test_completion_checklist.py
new file mode 100644
index 000000000..b13240e7d
--- /dev/null
+++ b/apps/job/tests/test_completion_checklist.py
@@ -0,0 +1,197 @@
+"""Tests for the front-desk completion checklist.
+
+The items are Job fields, so the audit trail comes from the job's own
+field-change machinery. These tests pin that each tick is attributed and that
+ticking nothing still lets a job be invoiced — the checklist records, it does
+not gate.
+"""
+
+import uuid
+from datetime import date
+from decimal import Decimal
+
+from django.urls import reverse
+from django.utils import timezone
+from rest_framework.response import Response
+
+from apps.accounting.services.invoice_calculation import calculate_invoice_amount
+from apps.accounts.models import Staff
+from apps.company.models import Company
+from apps.job.models import Job, JobEvent
+from apps.job.models.costing import CostLine
+from apps.testing import BaseAPITestCase
+
+CHECKLIST_EVENT = "completion_checklist_updated"
+
+
+class TestCompletionChecklist(BaseAPITestCase):
+    def setUp(self) -> None:
+        self.client_obj = Company.objects.create(
+            name="Test Company",
+            xero_last_modified=timezone.now(),
+        )
+        self.job = Job(
+            company=self.client_obj,
+            name="Test Job",
+            pricing_methodology="time_materials",
+        )
+        self.job.save(staff=self.test_staff)
+        self.url = reverse("jobs:job_finish_rest", args=[self.job.id])
+        # Ticking an item is an office action; the shared test_staff is
+        # deliberately neither office nor workshop.
+        self.office_staff = Staff.objects.create_user(
+            email="office@example.com",
+            password="testpass",
+            first_name="Office",
+            last_name="Person",
+            is_office_staff=True,
+        )
+        self.client.force_authenticate(user=self.office_staff)
+
+    def _patch(self, payload: dict[str, object]) -> Response:
+        return self.client.patch(self.url, data=payload, format="json")
+
+    # --- Reading ---
+
+    def test_a_new_job_has_nothing_ticked(self) -> None:
+        response = self.client.get(self.url)
+
+        self.assertEqual(response.status_code, 200)
+        for field in Job.COMPLETION_CHECKLIST_FIELDS:
+            self.assertFalse(response.data["checklist"][field], msg=field)
+
+    def test_every_item_is_exposed(self) -> None:
+        """The API shape and the field tuple must not drift apart."""
+        response = self.client.get(self.url)
+
+        self.assertEqual(
+            set(response.data["checklist"].keys()), set(Job.COMPLETION_CHECKLIST_FIELDS)
+        )
+
+    # --- Updating ---
+
+    def test_a_missing_job_is_not_found(self) -> None:
+        """A bad job id is a 404, not the 500 the old handler produced."""
+        missing = reverse("jobs:job_finish_rest", args=[uuid.uuid4()])
+
+        self.assertEqual(self.client.get(missing).status_code, 404)
+        self.assertEqual(
+            self.client.patch(
+                missing, data={"released": True}, format="json"
+            ).status_code,
+            404,
+        )
+
+    def test_ticking_one_item_leaves_the_others_alone(self) -> None:
+        response = self._patch({"materials_checked": True})
+
+        self.assertEqual(response.status_code, 200)
+        self.assertTrue(response.data["checklist"]["materials_checked"])
+        self.assertFalse(response.data["checklist"]["foreman_signed_off"])
+        self.assertFalse(response.data["checklist"]["released"])
+
+    def test_every_item_can_be_ticked(self) -> None:
+        for field in Job.COMPLETION_CHECKLIST_FIELDS:
+            with self.subTest(field=field):
+                response = self._patch({field: True})
+                self.assertEqual(response.status_code, 200)
+                self.assertTrue(response.data["checklist"][field])
+
+    def test_a_tick_can_be_withdrawn(self) -> None:
+        self._patch({"released": True})
+
+        response = self._patch({"released": False})
+
+        self.assertFalse(response.data["checklist"]["released"])
+
+    def test_unknown_item_is_rejected(self) -> None:
+        response = self._patch({"everything_is_fine": True})
+
+        self.assertEqual(response.status_code, 400)
+
+    def test_a_value_that_is_not_a_boolean_is_rejected(self) -> None:
+        response = self._patch({"materials_checked": "maybe"})
+
+        self.assertEqual(response.status_code, 400)
+
+    def test_a_rejected_payload_applies_none_of_it(self) -> None:
+        """An unknown key fails the whole payload rather than half-applying it."""
+        response = self._patch({"materials_checked": True, "nonsense": True})
+
+        self.assertEqual(response.status_code, 400)
+        self.job.refresh_from_db()
+        self.assertFalse(self.job.materials_checked)
+
+    # --- Audit ---
+
+    def test_each_changed_item_is_attributed_in_job_history(self) -> None:
+        self._patch({"foreman_signed_off": True})
+
+        event = JobEvent.objects.filter(
+            job=self.job, event_type=CHECKLIST_EVENT
+        ).latest("timestamp")
+        self.assertEqual(event.staff, self.office_staff)
+        self.assertEqual(event.description, "Foreman signed the job off")
+
+    def test_withdrawing_a_tick_is_audited(self) -> None:
+        self._patch({"released": True})
+        self._patch({"released": False})
+
+        events = JobEvent.objects.filter(
+            job=self.job, event_type=CHECKLIST_EVENT
+        ).order_by("timestamp")
+        self.assertEqual(
+            [e.description for e in events], ["Job released", "Job release withdrawn"]
+        )
+
+    def test_reticking_the_same_value_adds_no_event(self) -> None:
+        self._patch({"materials_checked": True})
+        self._patch({"materials_checked": True})
+
+        self.assertEqual(
+            JobEvent.objects.filter(job=self.job, event_type=CHECKLIST_EVENT).count(), 1
+        )
+
+    # --- The checklist records, it does not gate ---
+
+    def test_ticking_everything_does_not_change_job_status(self) -> None:
+        original_status = self.job.status
+
+        for field in Job.COMPLETION_CHECKLIST_FIELDS:
+            self._patch({field: True})
+
+        self.job.refresh_from_db()
+        self.assertEqual(self.job.status, original_status)
+
+    def test_an_untouched_checklist_does_not_block_invoicing(self) -> None:
+        self._add_actual_revenue(Decimal("500"))
+
+        result = calculate_invoice_amount(self.job, mode="invoice_costs_to_date")
+
+        self.assertEqual(result.calculated_amount, Decimal("500"))
+
+    def test_ticks_do_not_change_the_invoice_amount(self) -> None:
+        self._add_actual_revenue(Decimal("500"))
+        before = calculate_invoice_amount(
+            self.job, mode="invoice_costs_to_date"
+        ).calculated_amount
+
+        self._patch({"timesheets_collected": True})
+        self._patch({"materials_checked": True})
+
+        self.job.refresh_from_db()
+        after = calculate_invoice_amount(
+            self.job, mode="invoice_costs_to_date"
+        ).calculated_amount
+        self.assertEqual(before, after)
+
+    def _add_actual_revenue(self, revenue: Decimal) -> None:
+        CostLine.objects.create(
+            cost_set=self.job.latest_actual,
+            kind="adjust",
+            desc="Test line",
+            quantity=Decimal("1.000"),
+            unit_cost=Decimal("0.00"),
+            unit_rev=revenue,
+            accounting_date=date.today(),
+        )
diff --git a/apps/job/tests/test_job_event_tracking.py b/apps/job/tests/test_job_event_tracking.py
index 3103b29f8..d5591eabf 100644
--- a/apps/job/tests/test_job_event_tracking.py
+++ b/apps/job/tests/test_job_event_tracking.py
@@ -325,16 +325,36 @@ def test_friendly_boolean_paid(self) -> None:
         )
         self.assertEqual(event.description, "Marked as paid")
 
-    def test_friendly_boolean_collected(self) -> None:
+    def test_friendly_checklist_confirmation(self) -> None:
         event = self._event(
-            "job_collected",
+            "completion_checklist_updated",
             {
                 "changes": [
-                    {"field_name": "Collected", "old_value": "No", "new_value": "Yes"}
+                    {
+                        "field_name": "Foreman sign-off",
+                        "old_value": "No",
+                        "new_value": "Yes",
+                    }
+                ]
+            },
+        )
+        self.assertEqual(event.description, "Foreman signed the job off")
+
+    def test_friendly_checklist_withdrawal(self) -> None:
+        """Untickings are the changes worth finding later, so they read plainly."""
+        event = self._event(
+            "completion_checklist_updated",
+            {
+                "changes": [
+                    {
+                        "field_name": "Job released",
+                        "old_value": "Yes",
+                        "new_value": "No",
+                    }
                 ]
             },
         )
-        self.assertEqual(event.description, "Marked as collected")
+        self.assertEqual(event.description, "Job release withdrawn")
 
     def test_long_text_field_truncates(self) -> None:
         long_value = "x" * 200
diff --git a/apps/job/tests/test_mcp_tool_integration.py b/apps/job/tests/test_mcp_tool_integration.py
index 7f5b61845..cfd958cbc 100644
--- a/apps/job/tests/test_mcp_tool_integration.py
+++ b/apps/job/tests/test_mcp_tool_integration.py
@@ -4,7 +4,6 @@
 Tests cover:
 - QuotingTool functionality
 - SupplierProductQueryTool functionality
-- Tool parameter validation
 - Error handling in tool execution
 """
 
@@ -114,10 +113,6 @@ def setUp(self) -> None:
 
         self.tool = QuotingTool()
 
-    def test_tool_initialization(self) -> None:
-        """Test tool initializes correctly"""
-        self.assertIsInstance(self.tool, QuotingTool)
-
     def test_search_products_basic(self) -> None:
         """Test basic product search functionality"""
         result = self.tool.search_products(query="steel")
@@ -277,19 +272,6 @@ def setUp(self) -> None:
 
         self.tool = SupplierProductQueryTool()
 
-    def test_tool_initialization(self) -> None:
-        """Test tool initializes correctly"""
-        self.assertIsInstance(self.tool, SupplierProductQueryTool)
-
-    def test_model_attribute(self) -> None:
-        """Test that model is set to SupplierProduct"""
-        self.assertEqual(self.tool.model, SupplierProduct)
-
-    def test_exclude_fields(self) -> None:
-        """Test that supplier and price_list are excluded"""
-        self.assertIn("supplier", self.tool.exclude_fields)
-        self.assertIn("price_list", self.tool.exclude_fields)
-
     def test_get_queryset(self) -> None:
         """Test that get_queryset returns products with related data"""
         queryset = self.tool.get_queryset()
@@ -324,14 +306,6 @@ def setUp(self) -> None:
             staff=self.test_staff,
         )
 
-    def test_tool_parameter_validation(self) -> None:
-        """Test tool parameter validation"""
-        tool = QuotingTool()
-
-        # Test missing required parameters
-        with self.assertRaises(TypeError):
-            tool.search_products()  # Missing query parameter
-
     def test_tool_response_is_string(self) -> None:
         """Test that tools return string responses"""
         tool = QuotingTool()
diff --git a/apps/job/urls_rest.py b/apps/job/urls_rest.py
index b2a7a4e26..454d40e12 100644
--- a/apps/job/urls_rest.py
+++ b/apps/job/urls_rest.py
@@ -37,6 +37,7 @@
     JobDetailRestView,
     JobEventListRestView,
     JobEventRestView,
+    JobFinishRestView,
     JobHeaderRestView,
     JobInvoicesRestView,
     JobQuoteAcceptRestView,
@@ -176,6 +177,12 @@
         JobCostSummaryRestView.as_view(),
         name="job_cost_summary_rest",
     ),
+    # Finish Job workspace: customer balance + completion checklist
+    path(
+        "jobs/<uuid:job_id>/finish/",
+        JobFinishRestView.as_view(),
+        name="job_finish_rest",
+    ),
     # Job status choices
     path(
         "jobs/status-choices/",
diff --git a/apps/job/views/__init__.py b/apps/job/views/__init__.py
index c1a210fbf..258a7c582 100644
--- a/apps/job/views/__init__.py
+++ b/apps/job/views/__init__.py
@@ -44,6 +44,7 @@
     JobDetailRestView,
     JobEventListRestView,
     JobEventRestView,
+    JobFinishRestView,
     JobHeaderRestView,
     JobInvoicesRestView,
     JobQuoteAcceptRestView,
@@ -137,6 +138,7 @@
     "JobFileDetailView",
     "JobFileThumbnailView",
     "JobFilesCollectionView",
+    "JobFinishRestView",
     "JobHeaderRestView",
     "JobInvoicesRestView",
     "JobLabourRatesView",
diff --git a/apps/job/views/job_rest_views.py b/apps/job/views/job_rest_views.py
index 1ffdcf751..bc96dbb28 100644
--- a/apps/job/views/job_rest_views.py
+++ b/apps/job/views/job_rest_views.py
@@ -22,16 +22,23 @@
 from rest_framework import status
 from rest_framework.decorators import api_view, permission_classes
 from rest_framework.permissions import SAFE_METHODS, IsAuthenticated
+from rest_framework.request import Request
 from rest_framework.response import Response
 from rest_framework.views import APIView
 
+from apps.accounting.services.finish_job_summary import build_finish_job_summary
+from apps.accounting.services.invoice_calculation import (
+    get_job_for_invoice_calculation,
+)
 from apps.accounts.models import Staff
 from apps.job.etag import generate_job_etag
 from apps.job.models import Job, JobDeltaRejection
 from apps.job.permissions import IsOfficeStaff
 from apps.job.serializers.job_serializer import (
     CompanyDefaultsJobDetailSerializer,
+    FinishJobPayload,
     JobBasicInformationResponseSerializer,
+    JobCompletionChecklistUpdateSerializer,
     JobCostSummaryResponseSerializer,
     JobCreateResponseSerializer,
     JobCreateSerializer,
@@ -42,6 +49,7 @@
     JobEventCreateResponseSerializer,
     JobEventCreateSerializer,
     JobEventsResponseSerializer,
+    JobFinishResponseSerializer,
     JobHeaderResponseSerializer,
     JobInvoicesResponseSerializer,
     JobQuoteAcceptanceSerializer,
@@ -57,6 +65,7 @@
     DeltaValidationError,
     JobRestService,
 )
+from apps.job.services.job_service import update_completion_checklist
 from apps.workflow.etag import if_none_match_satisfied
 from apps.workflow.exceptions import PreconditionFailedError
 from apps.workflow.models import CompanyDefaults
@@ -863,8 +872,6 @@ def get(self, request, job_id):
             resp = self._set_etag(resp, current_etag)
             return resp
 
-        except Job.DoesNotExist as exc:
-            raise ValueError(f"Job with id {job_id} not found") from exc
         except Exception as e:
             return self.handle_service_error(e)
 
@@ -905,8 +912,6 @@ def get(self, request, job_id):
             resp = Response(serializer.data, status=status.HTTP_200_OK)
             return self._set_etag(resp, current_etag)
 
-        except Job.DoesNotExist as exc:
-            raise ValueError(f"Job with id {job_id} not found") from exc
         except Exception as e:
             return self.handle_service_error(e)
 
@@ -946,8 +951,6 @@ def get(self, request, job_id):
             resp = Response(quote, status=status.HTTP_200_OK)
             return self._set_etag(resp, current_etag)
 
-        except Job.DoesNotExist as exc:
-            raise ValueError(f"Job with id {job_id} not found") from exc
         except Exception as e:
             return self.handle_service_error(e)
 
@@ -1072,8 +1075,80 @@ def calculate_profit_margin(cost_set):
             resp = Response(serializer.data, status=status.HTTP_200_OK)
             return self._set_etag(resp, current_etag)
 
-        except Job.DoesNotExist as exc:
-            raise ValueError(f"Job with id {job_id} not found") from exc
+        except Exception as e:
+            return self.handle_service_error(e)
+
+
+def _finish_job_payload(job: Job) -> FinishJobPayload:
+    return {
+        "summary": build_finish_job_summary(job),
+        "checklist": job,
+    }
+
+
+@method_decorator(csrf_exempt, name="dispatch")
+class JobFinishRestView(BaseJobRestView):
+    """Finish Job workspace: the authoritative customer balance and checklist.
+
+    Deliberately not ETagged on the job. The balance moves when invoices change,
+    and an invoice arriving from Xero does not touch ``job.updated_at``, so a
+    job-derived ETag would answer 304 while showing a customer the wrong amount
+    to pay.
+    """
+
+    serializer_class = JobFinishResponseSerializer
+
+    @extend_schema(
+        responses={
+            200: JobFinishResponseSerializer,
+            400: JobRestErrorResponseSerializer,
+        },
+        description=(
+            "Fetch the authoritative Finish Job customer balance and completion "
+            "checklist for a job. All currency values are calculated server-side."
+        ),
+        tags=["Jobs"],
+    )
+    def get(self, request: Request, job_id: UUID) -> Response:
+        try:
+            job = get_job_for_invoice_calculation(job_id)
+            serializer = JobFinishResponseSerializer(_finish_job_payload(job))
+            return Response(serializer.data, status=status.HTTP_200_OK)
+
+        except Exception as e:
+            return self.handle_service_error(e)
+
+    @extend_schema(
+        request=JobCompletionChecklistUpdateSerializer,
+        responses={
+            200: JobFinishResponseSerializer,
+            400: JobRestErrorResponseSerializer,
+        },
+        description=(
+            "Update one or more completion checklist items. Each changed item adds "
+            "a job-history event. Unknown item keys are rejected."
+        ),
+        tags=["Jobs"],
+    )
+    def patch(self, request: Request, job_id: UUID) -> Response:
+        try:
+            # A tick records who made it, so narrow the authenticated user to a
+            # real Staff rather than casting (ADR 0028).
+            staff = request.user
+            if not isinstance(staff, Staff):
+                raise ValueError(
+                    "Checklist updates require an authenticated staff member."
+                )
+
+            updates = JobCompletionChecklistUpdateSerializer(data=request.data)
+            updates.is_valid(raise_exception=True)
+
+            job = update_completion_checklist(
+                get_job_for_invoice_calculation(job_id), updates.validated_data, staff
+            )
+            serializer = JobFinishResponseSerializer(_finish_job_payload(job))
+            return Response(serializer.data, status=status.HTTP_200_OK)
+
         except Exception as e:
             return self.handle_service_error(e)
 
@@ -1141,8 +1216,6 @@ def get(self, request, job_id):
             serializer = JobEventsResponseSerializer({"events": events})
             return Response(serializer.data, status=status.HTTP_200_OK)
 
-        except Job.DoesNotExist as exc:
-            raise ValueError(f"Job with id {job_id} not found") from exc
         except Exception as e:
             return self.handle_service_error(e)
 
@@ -1382,11 +1455,8 @@ def get(self, request, job_id):
         """
         try:
             # Conditional GET using ETag based on Job.updated_at
-            try:
-                job = Job.objects.only("id", "updated_at").get(id=job_id)
-                current_etag = self._gen_job_etag(job)
-            except Job.DoesNotExist as exc:
-                raise ValueError(f"Job with id {job_id} not found") from exc
+            job = Job.objects.only("id", "updated_at").get(id=job_id)
+            current_etag = self._gen_job_etag(job)
 
             if_none_match = self._get_if_none_match(request)
             if if_none_match and if_none_match_satisfied(if_none_match, current_etag):
diff --git a/apps/workflow/accounting/xero/readonly_provider.py b/apps/workflow/accounting/xero/readonly_provider.py
index 82485f537..cb368174c 100644
--- a/apps/workflow/accounting/xero/readonly_provider.py
+++ b/apps/workflow/accounting/xero/readonly_provider.py
@@ -34,7 +34,6 @@
 
 logger = logging.getLogger("xero")
 
-_GST_RATE = Decimal("0.15")
 _CENT = Decimal("0.01")
 
 
@@ -48,10 +47,12 @@ def _log_suppressed(operation: str, detail: str) -> None:
 
 def _fake_totals(line_items: list[DocumentLineItem]) -> tuple[str, str, str]:
     """Cosmetic GST-exclusive totals for stubbed documents (local display only)."""
+    from apps.workflow.models import CompanyDefaults
+
     sub_total = sum(
         (li.quantity * li.unit_amount for li in line_items), Decimal("0")
     ).quantize(_CENT)
-    tax = (sub_total * _GST_RATE).quantize(_CENT)
+    tax = (sub_total * CompanyDefaults.get_solo().gst_rate).quantize(_CENT)
     total = sub_total + tax
     return str(sub_total), str(tax), str(total)
 
diff --git a/apps/workflow/migrations/0019_company_defaults_gst_rate.py b/apps/workflow/migrations/0019_company_defaults_gst_rate.py
new file mode 100644
index 000000000..f9cec3ce4
--- /dev/null
+++ b/apps/workflow/migrations/0019_company_defaults_gst_rate.py
@@ -0,0 +1,25 @@
+# Generated by Django 6.0.7 on 2026-07-31 21:21
+
+from decimal import Decimal
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("workflow", "0018_text_unset_constraints"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="companydefaults",
+            name="gst_rate",
+            field=models.DecimalField(
+                decimal_places=4,
+                default=Decimal("0.1500"),
+                help_text="Sales tax rate applied to amounts DocketWorks quotes before Xero has issued an invoice, as a fraction (0.1500 = 15% NZ GST). Xero remains authoritative for tax on invoices that exist.",
+                max_digits=5,
+            ),
+        ),
+    ]
diff --git a/apps/workflow/migrations/0020_company_defaults_gst_rate_bounds.py b/apps/workflow/migrations/0020_company_defaults_gst_rate_bounds.py
new file mode 100644
index 000000000..42447e083
--- /dev/null
+++ b/apps/workflow/migrations/0020_company_defaults_gst_rate_bounds.py
@@ -0,0 +1,38 @@
+# Generated by Django 6.0.7 on 2026-08-01 07:24
+
+from decimal import Decimal
+
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("company", "0012_text_unset_constraints"),
+        ("workflow", "0019_company_defaults_gst_rate"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="companydefaults",
+            name="gst_rate",
+            field=models.DecimalField(
+                decimal_places=4,
+                default=Decimal("0.1500"),
+                help_text="Sales tax rate applied to amounts DocketWorks quotes before Xero has issued an invoice, as a fraction (0.1500 = 15% NZ GST). Xero remains authoritative for tax on invoices that exist.",
+                max_digits=5,
+                validators=[
+                    django.core.validators.MinValueValidator(Decimal("0")),
+                    django.core.validators.MaxValueValidator(Decimal("0.9999")),
+                ],
+            ),
+        ),
+        migrations.AddConstraint(
+            model_name="companydefaults",
+            constraint=models.CheckConstraint(
+                condition=models.Q(("gst_rate__gte", 0), ("gst_rate__lt", 1)),
+                name="companydefaults_gst_rate_is_a_fraction",
+            ),
+        ),
+    ]
diff --git a/apps/workflow/models/company_defaults.py b/apps/workflow/models/company_defaults.py
index 9fc58ce54..9dfbc9ed0 100644
--- a/apps/workflow/models/company_defaults.py
+++ b/apps/workflow/models/company_defaults.py
@@ -1,6 +1,7 @@
 from collections.abc import Iterable
 from decimal import Decimal
 
+from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
 from django.db.models.base import ModelBase
 from solo.models import SingletonModel
@@ -20,6 +21,20 @@ class CompanyDefaults(SingletonModel):
     )
     time_markup = models.DecimalField(max_digits=5, decimal_places=2, default=0.3)
     materials_markup = models.DecimalField(max_digits=5, decimal_places=2, default=0.2)
+    gst_rate = models.DecimalField(
+        max_digits=5,
+        decimal_places=4,
+        default=Decimal("0.1500"),
+        validators=[
+            MinValueValidator(Decimal("0")),
+            MaxValueValidator(Decimal("0.9999")),
+        ],
+        help_text=(
+            "Sales tax rate applied to amounts DocketWorks quotes before Xero has "
+            "issued an invoice, as a fraction (0.1500 = 15% NZ GST). Xero remains "
+            "authoritative for tax on invoices that exist."
+        ),
+    )
     wage_rate = models.DecimalField(
         max_digits=6, decimal_places=2, default=32.00
     )  # rate per hour
@@ -358,6 +373,13 @@ class Meta:
                 condition=models.Q(id=1),
                 name="companydefaults_singleton",
             ),
+            # The validators above only fire through a serializer; the admin,
+            # management commands and the Xero sync all write straight to the
+            # model, so the rate is held to a fraction at the database.
+            models.CheckConstraint(
+                condition=models.Q(gst_rate__gte=0) & models.Q(gst_rate__lt=1),
+                name="companydefaults_gst_rate_is_a_fraction",
+            ),
         ]
 
     @classmethod
diff --git a/apps/workflow/models/settings_metadata.py b/apps/workflow/models/settings_metadata.py
index 15b85b593..79e49bf4e 100644
--- a/apps/workflow/models/settings_metadata.py
+++ b/apps/workflow/models/settings_metadata.py
@@ -111,6 +111,7 @@ def get_section_info(cls, key: str) -> tuple[str, str, int] | None:
     # Finances
     "time_markup": "finances",
     "materials_markup": "finances",
+    "gst_rate": "finances",
     "wage_rate": "finances",
     "annual_leave_loading": "finances",
     "financial_year_start_month": "finances",
diff --git a/apps/workflow/tests/test_access_logging_middleware.py b/apps/workflow/tests/test_access_logging_middleware.py
index a44959f8f..8cb48177e 100644
--- a/apps/workflow/tests/test_access_logging_middleware.py
+++ b/apps/workflow/tests/test_access_logging_middleware.py
@@ -29,10 +29,7 @@ def test_access_log_includes_response_status_and_duration() -> None:
 
     assert response.status_code == 418
     log_info.assert_called_once()
-    log_format, *log_args = log_info.call_args.args
-    assert log_format == (
-        "%s\tmethod=%s\tstatus=%s\tduration_ms=%.2f\treplay=%s\tuser=%s\tpath=%s"
-    )
+    _, *log_args = log_info.call_args.args
     assert log_args[1] == "GET"
     assert log_args[2] == 418
     assert log_args[3] == pytest.approx(123.45)
diff --git a/apps/workflow/tests/test_backup_scripts.py b/apps/workflow/tests/test_backup_scripts.py
index 72f885eaf..d0d31b991 100644
--- a/apps/workflow/tests/test_backup_scripts.py
+++ b/apps/workflow/tests/test_backup_scripts.py
@@ -16,7 +16,6 @@
 
 REPO_ROOT = Path(__file__).resolve().parents[3]
 CLEANUP_BACKUPS = REPO_ROOT / "scripts" / "cleanup_backups.py"
-BACKUP_INSTANCE_FILES = REPO_ROOT / "scripts" / "backup_instance_files.sh"
 PULL_PROD_BACKUP = REPO_ROOT / "scripts" / "pull_prod_backup.sh"
 
 
@@ -241,7 +240,6 @@ def purge_remote_entries(
             self.assertEqual(events[0][0], "copy")
             self.assertIn("daily_20260601.sql.gz", events[0][1])
             self.assertEqual(events[1], ("purge", ["daily_20260601.sql.gz"]))
-            self.assertEqual(cleanup.REMOTE_BASE, "gdrive:dw_backups")
             self.assertFalse((backup_dir / "daily_20260601.sql.gz").exists())
 
     def test_cleanup_copy_failure_prevents_local_retention_delete(self) -> None:
@@ -324,22 +322,6 @@ def test_cleanup_prunes_daily_sha_with_expired_dump(self) -> None:
             self.assertFalse((backup_dir / "daily_20260601.sha").exists())
             self.assertTrue((backup_dir / "daily_20260615.sha").exists())
 
-    def test_file_backup_script_is_incremental_and_scoped(self) -> None:
-        content = BACKUP_INSTANCE_FILES.read_text()
-
-        self.assertIn('backup_dir "phone-recordings"', content)
-        self.assertIn('backup_dir "session-replays"', content)
-        self.assertIn('backup_dir "mediafiles"', content)
-        self.assertNotIn('backup_dir "dropbox"', content)
-        self.assertNotIn('backup_dir "adhoc"', content)
-        self.assertNotIn("tar ", content)
-        self.assertIn("rclone sync", content)
-        self.assertIn("--backup-dir", content)
-        self.assertIn('REMOTE_BASE="gdrive:dw_backups/files"', content)
-        self.assertIn("ARCHIVE_RETENTION_DAYS=30", content)
-        self.assertIn("rclone purge", content)
-        self.assertIn("refusing to back up symlinked directory", content)
-
 
 class BackportCommandErrorPersistenceTests(TestCase):
     def test_prelogged_scrub_failure_is_not_persisted_again(self) -> None:
diff --git a/apps/workflow/tests/test_company_defaults_api.py b/apps/workflow/tests/test_company_defaults_api.py
index 75cc22506..2dbb02374 100644
--- a/apps/workflow/tests/test_company_defaults_api.py
+++ b/apps/workflow/tests/test_company_defaults_api.py
@@ -1,6 +1,7 @@
 import uuid
+from decimal import Decimal
 
-from django.db import connection
+from django.db import IntegrityError, connection, transaction
 from django.test.utils import CaptureQueriesContext
 from rest_framework import status
 from rest_framework.test import APIClient
@@ -167,3 +168,24 @@ def test_patch_rejects_xero_quote_terms_over_4000_characters(self) -> None:
             payload["xero_quote_terms"],
             ["Ensure this field has no more than 4000 characters."],
         )
+
+    def test_patch_rejects_a_gst_rate_outside_a_fraction(self) -> None:
+        """The rate is a fraction: 0.15 means 15%, so 15 would tax 1500%."""
+        for rate in (Decimal("-0.1500"), Decimal("15")):
+            with self.subTest(rate=rate):
+                response = self.client.patch(
+                    "/api/company-defaults/", {"gst_rate": rate}, format="json"
+                )
+
+                self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+                self.assertIn("gst_rate", response.json())
+
+    def test_a_gst_rate_outside_a_fraction_cannot_reach_the_database(self) -> None:
+        """The admin, commands and Xero sync all bypass the serializer."""
+        defaults = CompanyDefaults.get_solo()
+
+        for rate in (Decimal("-0.1500"), Decimal("1.0000")):
+            with self.subTest(rate=rate):
+                defaults.gst_rate = rate
+                with self.assertRaises(IntegrityError), transaction.atomic():
+                    defaults.save()
diff --git a/docs/urls/job.md b/docs/urls/job.md
index 4d1062d4e..3047a1f2c 100644
--- a/docs/urls/job.md
+++ b/docs/urls/job.md
@@ -52,6 +52,7 @@
 | `/jobs/<uuid:job_id>/files/` | `job_files_collection_view.JobFilesCollectionView` | `jobs:job_files_collection` | Collection operations on job files. |
 | `/jobs/<uuid:job_id>/files/<uuid:file_id>/` | `job_file_detail_view.JobFileDetailView` | `jobs:job_file_detail` | Resource operations on individual job files. |
 | `/jobs/<uuid:job_id>/files/<uuid:file_id>/thumbnail/` | `job_file_thumbnail_view.JobFileThumbnailView` | `jobs:job_file_thumbnail` | Thumbnail serving for job files. |
+| `/jobs/<uuid:job_id>/finish/` | `job_rest_views.JobFinishRestView` | `jobs:job_finish_rest` | Finish Job workspace: the authoritative customer balance and checklist. |
 | `/jobs/<uuid:job_id>/header/` | `job_rest_views.JobHeaderRestView` | `jobs:job_header_rest` | REST view for Job header information. |
 | `/jobs/<uuid:job_id>/invoices/` | `job_rest_views.JobInvoicesRestView` | `jobs:job_invoices_rest` | REST view for Job invoices. |
 | `/jobs/<uuid:job_id>/labour-rates/` | `labour_views.JobLabourRatesView` | `jobs:job_labour_rates_rest` | Read or update a job's per-subtype charge-out rates. |
diff --git a/frontend/schema.yml b/frontend/schema.yml
index 94c814cf9..93bfbbd02 100644
--- a/frontend/schema.yml
+++ b/frontend/schema.yml
@@ -3996,6 +3996,74 @@ paths:
               schema:
                 $ref: '#/components/schemas/JobFileThumbnailErrorResponse'
           description: ''
+  /api/job/jobs/{job_id}/finish/:
+    get:
+      operationId: job_jobs_finish_retrieve
+      description: Fetch the authoritative Finish Job customer balance and completion
+        checklist for a job. All currency values are calculated server-side.
+      parameters:
+        - in: path
+          name: job_id
+          schema:
+            type: string
+            format: uuid
+          required: true
+      tags:
+        - Jobs
+      security:
+        - cookieAuth: []
+      responses:
+        '200':
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/JobFinishResponse'
+          description: ''
+        '400':
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/JobRestErrorResponse'
+          description: ''
+    patch:
+      operationId: job_jobs_finish_partial_update
+      description: Update one or more completion checklist items. Each changed item
+        adds a job-history event. Unknown item keys are rejected.
+      parameters:
+        - in: path
+          name: job_id
+          schema:
+            type: string
+            format: uuid
+          required: true
+      tags:
+        - Jobs
+      requestBody:
+        content:
+          application/json:
+            schema:
+              $ref: '#/components/schemas/PatchedJobCompletionChecklistUpdateRequest'
+          application/x-www-form-urlencoded:
+            schema:
+              $ref: '#/components/schemas/PatchedJobCompletionChecklistUpdateRequest'
+          multipart/form-data:
+            schema:
+              $ref: '#/components/schemas/PatchedJobCompletionChecklistUpdateRequest'
+      security:
+        - cookieAuth: []
+      responses:
+        '200':
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/JobFinishResponse'
+          description: ''
+        '400':
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/JobRestErrorResponse'
+          description: ''
   /api/job/jobs/{job_id}/header/:
     get:
       operationId: job_jobs_header_retrieve
@@ -10796,6 +10864,14 @@ components:
           minimum: -1000
           exclusiveMaximum: true
           exclusiveMinimum: true
+        gst_rate:
+          type: number
+          format: double
+          maximum: 0.9999
+          minimum: 0
+          description: Sales tax rate applied to amounts DocketWorks quotes before
+            Xero has issued an invoice, as a fraction (0.1500 = 15% NZ GST). Xero
+            remains authoritative for tax on invoices that exist.
         wage_rate:
           type: number
           format: double
@@ -11184,6 +11260,14 @@ components:
           minimum: -1000
           exclusiveMaximum: true
           exclusiveMinimum: true
+        gst_rate:
+          type: number
+          format: double
+          maximum: 0.9999
+          minimum: 0
+          description: Sales tax rate applied to amounts DocketWorks quotes before
+            Xero has issued an invoice, as a fraction (0.1500 = 15% NZ GST). Xero
+            remains authoritative for tax on invoices that exist.
         wage_rate:
           type: number
           format: double
@@ -13262,6 +13346,88 @@ components:
       required:
         - end_date
         - start_date
+    FinishJobSummary:
+      type: object
+      description: |-
+        The authoritative customer balance shown in the Finish Job workspace.
+
+        Read-only: every value is calculated by
+        apps.accounting.services.finish_job_summary, and the frontend formats rather
+        than recomputes them (ADR 0020).
+      properties:
+        job_value_excl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        valid_invoiced_excl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        outstanding_invoiced_incl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        remaining_to_invoice_excl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        remaining_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        remaining_to_invoice_incl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        total_to_pay_incl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+        over_invoiced_excl_gst:
+          type: number
+          format: double
+          maximum: 10000000000
+          minimum: -10000000000
+          exclusiveMaximum: true
+          exclusiveMinimum: true
+          readOnly: true
+      required:
+        - job_value_excl_gst
+        - outstanding_invoiced_incl_gst
+        - over_invoiced_excl_gst
+        - remaining_gst
+        - remaining_to_invoice_excl_gst
+        - remaining_to_invoice_incl_gst
+        - total_to_pay_incl_gst
+        - valid_invoiced_excl_gst
     FormCreateRequest:
       type: object
       description: Request serializer for creating a form document (no Google Doc).
@@ -14153,6 +14319,37 @@ components:
         - job_name
         - job_number
         - revenue
+    JobCompletionChecklist:
+      type: object
+      description: |-
+        Read shape for the front-desk completion checklist.
+
+        The items are Job fields, so each tick is audited by the job's own
+        field-change machinery. Who ticked what, and when, is in the job history.
+      properties:
+        foreman_signed_off:
+          type: boolean
+          readOnly: true
+        timesheets_collected:
+          type: boolean
+          readOnly: true
+        materials_checked:
+          type: boolean
+          readOnly: true
+        customer_called:
+          type: boolean
+          readOnly: true
+        released:
+          type: boolean
+          readOnly: true
+          description: The job has left our possession, by collection, delivery or
+            on-site install.
+      required:
+        - customer_called
+        - foreman_signed_off
+        - materials_checked
+        - released
+        - timesheets_collected
     JobCostSetSummary:
       type: object
       description: Serializer for cost set summary data in job views
@@ -14678,6 +14875,21 @@ components:
       required:
         - message
         - uploaded
+    JobFinishResponse:
+      type: object
+      description: Everything the Finish Job workspace reads in one request.
+      properties:
+        summary:
+          allOf:
+            - $ref: '#/components/schemas/FinishJobSummary'
+          readOnly: true
+        checklist:
+          allOf:
+            - $ref: '#/components/schemas/JobCompletionChecklist'
+          readOnly: true
+      required:
+        - checklist
+        - summary
     JobForPurchasing:
       type: object
       description: Serializer for Job model in purchasing contexts.
@@ -17100,6 +17312,14 @@ components:
           minimum: -1000
           exclusiveMaximum: true
           exclusiveMinimum: true
+        gst_rate:
+          type: number
+          format: double
+          maximum: 0.9999
+          minimum: 0
+          description: Sales tax rate applied to amounts DocketWorks quotes before
+            Xero has issued an invoice, as a fraction (0.1500 = 15% NZ GST). Xero
+            remains authoritative for tax on invoices that exist.
         wage_rate:
           type: number
           format: double
@@ -17585,6 +17805,26 @@ components:
           description: JSON schema defining entry fields for form templates
         status:
           $ref: '#/components/schemas/FormStatusEnum'
+    PatchedJobCompletionChecklistUpdateRequest:
+      type: object
+      description: |-
+        Partial update shape: send only the items being changed.
+
+        Unknown keys are rejected rather than dropped, so a client typo is a 400
+        instead of a silent no-op.
+      properties:
+        foreman_signed_off:
+          type: boolean
+        timesheets_collected:
+          type: boolean
+        materials_checked:
+          type: boolean
+        customer_called:
+          type: boolean
+        released:
+          type: boolean
+          description: The job has left our possession, by collection, delivery or
+            on-site install.
     PatchedJobDeltaEnvelopeRequest:
       type: object
       description: Serializer that validates the delta envelope submitted by the frontend.
diff --git a/frontend/src/api/generated/api.ts b/frontend/src/api/generated/api.ts
index 2f53ab219..c2274ada9 100644
--- a/frontend/src/api/generated/api.ts
+++ b/frontend/src/api/generated/api.ts
@@ -896,6 +896,7 @@ const CompanyDefaults = z.object({
   company_acronym: z.string().max(10).nullish(),
   time_markup: z.number().gt(-1000).lt(1000).optional(),
   materials_markup: z.number().gt(-1000).lt(1000).optional(),
+  gst_rate: z.number().gte(0).lte(0.9999).optional(),
   wage_rate: z.number().gt(-10000).lt(10000).optional(),
   annual_leave_loading: z.number().gt(-1000).lt(1000).optional(),
   workshop_efficiency_factor: z.number().gt(-10).lt(10).optional(),
@@ -962,6 +963,7 @@ const CompanyDefaultsRequest = z.object({
   company_acronym: z.string().min(1).max(10).nullish(),
   time_markup: z.number().gt(-1000).lt(1000).optional(),
   materials_markup: z.number().gt(-1000).lt(1000).optional(),
+  gst_rate: z.number().gte(0).lte(0.9999).optional(),
   wage_rate: z.number().gt(-10000).lt(10000).optional(),
   annual_leave_loading: z.number().gt(-1000).lt(1000).optional(),
   workshop_efficiency_factor: z.number().gt(-10).lt(10).optional(),
@@ -1027,6 +1029,7 @@ const PatchedCompanyDefaultsRequest = z
     company_acronym: z.string().min(1).max(10).nullable(),
     time_markup: z.number().gt(-1000).lt(1000),
     materials_markup: z.number().gt(-1000).lt(1000),
+    gst_rate: z.number().gte(0).lte(0.9999),
     wage_rate: z.number().gt(-10000).lt(10000),
     annual_leave_loading: z.number().gt(-1000).lt(1000),
     workshop_efficiency_factor: z.number().gt(-10).lt(10),
@@ -1785,6 +1788,36 @@ const JobFileThumbnailErrorResponse = z.object({
   status: z.string().optional().default('error'),
   message: z.string(),
 })
+const FinishJobSummary = z.object({
+  job_value_excl_gst: z.number().gt(-10000000000).lt(10000000000),
+  valid_invoiced_excl_gst: z.number().gt(-10000000000).lt(10000000000),
+  outstanding_invoiced_incl_gst: z.number().gt(-10000000000).lt(10000000000),
+  remaining_to_invoice_excl_gst: z.number().gt(-10000000000).lt(10000000000),
+  remaining_gst: z.number().gt(-10000000000).lt(10000000000),
+  remaining_to_invoice_incl_gst: z.number().gt(-10000000000).lt(10000000000),
+  total_to_pay_incl_gst: z.number().gt(-10000000000).lt(10000000000),
+  over_invoiced_excl_gst: z.number().gt(-10000000000).lt(10000000000),
+})
+const JobCompletionChecklist = z.object({
+  foreman_signed_off: z.boolean(),
+  timesheets_collected: z.boolean(),
+  materials_checked: z.boolean(),
+  customer_called: z.boolean(),
+  released: z.boolean(),
+})
+const JobFinishResponse = z.object({
+  summary: FinishJobSummary,
+  checklist: JobCompletionChecklist,
+})
+const PatchedJobCompletionChecklistUpdateRequest = z
+  .object({
+    foreman_signed_off: z.boolean(),
+    timesheets_collected: z.boolean(),
+    materials_checked: z.boolean(),
+    customer_called: z.boolean(),
+    released: z.boolean(),
+  })
+  .partial()
 const JobStatusEnum = z.enum([
   'draft',
   'awaiting_approval',
@@ -3936,6 +3969,10 @@ export const schemas = {
   JobFileRequest,
   JobFileUpdateSuccessResponse,
   JobFileThumbnailErrorResponse,
+  FinishJobSummary,
+  JobCompletionChecklist,
+  JobFinishResponse,
+  PatchedJobCompletionChecklistUpdateRequest,
   JobStatusEnum,
   JobHeaderResponse,
   JobInvoicesResponse,
@@ -7032,6 +7069,53 @@ POST /job/rest/jobs/&lt;uuid:pk&gt;/quote/preview/`,
       },
     ],
   },
+  {
+    method: 'get',
+    path: '/api/job/jobs/:job_id/finish/',
+    alias: 'job_jobs_finish_retrieve',
+    description: `Fetch the authoritative Finish Job customer balance and completion checklist for a job. All currency values are calculated server-side.`,
+    requestFormat: 'json',
+    parameters: [
+      {
+        name: 'job_id',
+        type: 'Path',
+        schema: z.string().uuid(),
+      },
+    ],
+    response: JobFinishResponse,
+    errors: [
+      {
+        status: 400,
+        schema: JobRestErrorResponse,
+      },
+    ],
+  },
+  {
+    method: 'patch',
+    path: '/api/job/jobs/:job_id/finish/',
+    alias: 'job_jobs_finish_partial_update',
+    description: `Update one or more completion checklist items. Each changed item adds a job-history event. Unknown item keys are rejected.`,
+    requestFormat: 'json',
+    parameters: [
+      {
+        name: 'body',
+        type: 'Body',
+        schema: PatchedJobCompletionChecklistUpdateRequest,
+      },
+      {
+        name: 'job_id',
+        type: 'Path',
+        schema: z.string().uuid(),
+      },
+    ],
+    response: JobFinishResponse,
+    errors: [
+      {
+        status: 400,
+        schema: JobRestErrorResponse,
+      },
+    ],
+  },
   {
     method: 'get',
     path: '/api/job/jobs/:job_id/header/',
diff --git a/frontend/src/components/job/JobActualTab.vue b/frontend/src/components/job/JobActualTab.vue
index 18bcb768e..0f046f0cd 100644
--- a/frontend/src/components/job/JobActualTab.vue
+++ b/frontend/src/components/job/JobActualTab.vue
@@ -35,16 +35,6 @@
           <span class="text-[11px] uppercase tracking-wide text-slate-600">Time & Expenses</span>
           <strong class="tabular-nums text-slate-900">{{ formatCurrency(timeAndExpenses) }}</strong>
         </li>
-        <li class="h-10 px-3 rounded-lg border border-slate-200 bg-white flex items-center gap-2">
-          <span class="w-1.5 h-6 rounded-full bg-orange-500"></span>
-          <span class="text-[11px] uppercase tracking-wide text-slate-600">Invoiced</span>
-          <strong class="tabular-nums text-slate-900">{{ formatCurrency(invoiceTotal) }}</strong>
-        </li>
-        <li class="h-10 px-3 rounded-lg border border-slate-200 bg-white flex items-center gap-2">
-          <span class="w-1.5 h-6 rounded-full bg-violet-500"></span>
-          <span class="text-[11px] uppercase tracking-wide text-slate-600">To Be Invoiced</span>
-          <strong class="tabular-nums text-slate-900">{{ formatCurrency(toBeInvoiced) }}</strong>
-        </li>
       </ul>
     </div>
 
@@ -99,7 +89,7 @@
         </div>
       </main>
 
-      <!-- ASIDE (STICKY): Summary + Invoices -->
+      <!-- ASIDE (STICKY): Actual summary -->
       <aside class="space-y-4 lg:sticky lg:top-16 self-start">
         <div class="bg-white rounded-xl border border-slate-200">
           <div class="p-3 w-full">
@@ -114,136 +104,6 @@
             />
           </div>
         </div>
-
-        <div class="bg-white rounded-xl border border-slate-200">
-          <Card class="border-0 shadow-none overflow-hidden">
-            <CardHeader class="px-3 pt-3 pb-2">
-              <CardTitle>
-                Invoices <span class="text-slate-400 text-sm">({{ invoices.length }})</span>
-              </CardTitle>
-              <CardDescription>Manage invoices for this job.</CardDescription>
-            </CardHeader>
-
-            <CardContent class="p-0 pb-2">
-              <div
-                class="max-h-[min(12vh,20rem)] overflow-y-auto px-2"
-                style="scrollbar-gutter: stable"
-              >
-                <div v-if="invoices.length === 0" class="text-center py-6 text-gray-500">
-                  No invoices for this project
-                </div>
-
-                <ul v-else role="list" class="divide-y divide-slate-200 rounded-md bg-white">
-                  <li
-                    v-for="invoice in invoices"
-                    :key="invoice.id"
-                    class="px-3 py-1.5 hover:bg-slate-50 flex items-center gap-3"
-                  >
-                    <div class="min-w-0 flex-1">
-                      <div class="flex items-center gap-2">
-                        <span class="font-medium text-slate-900 text-sm leading-5">
-                          {{ invoice.number }}
-                        </span>
-                        <Badge
-                          :variant="invoice.status === 'PAID' ? 'default' : 'secondary'"
-                          class="text-[10px] px-1.5 py-0.5 rounded-full"
-                        >
-                          {{ invoice.status }}
-                        </Badge>
-                      </div>
-                      <div class="text-[11px] leading-4 text-slate-500">
-                        {{ formatDate(invoice.date) }}
-                      </div>
-                    </div>
-
-                    <!-- Total -->
-                    <div class="shrink-0 text-sm font-semibold tabular-nums text-slate-900">
-                      {{ formatCurrency(invoice.total_excl_tax) }}
-                    </div>
-
-                    <!-- Actions -->
-                    <div class="shrink-0 flex items-center gap-2">
-                      <Button
-                        variant="outline"
-                        size="icon"
-                        class="h-7 w-7"
-                        @click="goToInvoiceOnXero(invoice.online_url)"
-                        :disabled="!invoice.online_url"
-                      >
-                        <ExternalLink class="h-4 w-4" />
-                      </Button>
-                      <Button
-                        variant="destructive"
-                        size="icon"
-                        class="h-7 w-7"
-                        @click="deleteInvoiceOnXero(invoice.xero_id)"
-                        :disabled="!!deletingInvoiceId"
-                      >
-                        <svg
-                          v-if="deletingInvoiceId === invoice.xero_id"
-                          class="animate-spin h-4 w-4"
-                          viewBox="0 0 24 24"
-                        >
-                          <circle
-                            class="opacity-25"
-                            cx="12"
-                            cy="12"
-                            r="10"
-                            stroke="currentColor"
-                            stroke-width="4"
-                          />
-                          <path
-                            class="opacity-75"
-                            fill="currentColor"
-                            d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z"
-                          />
-                        </svg>
-                        <Trash2 v-else class="h-4 w-4" />
-                      </Button>
-                    </div>
-                  </li>
-                </ul>
-              </div>
-            </CardContent>
-
-            <!-- Invoice footer: visible always; disabled if paid -->
-            <div>
-              <div class="border-t border-slate-200"></div>
-              <CardFooter class="flex flex-col items-center gap-2 pt-4">
-                <button
-                  @click="createInvoice()"
-                  :disabled="isCreatingInvoice || !!props.paid || !xeroConnected"
-                  class="px-4 py-2 bg-orange-600 text-white rounded-md hover:bg-orange-700 focus:outline-none focus:ring-2 focus:ring-orange-500 disabled:opacity-50 flex items-center gap-2"
-                >
-                  <svg
-                    v-if="isCreatingInvoice"
-                    class="animate-spin -ml-1 mr-1 h-4 w-4"
-                    viewBox="0 0 24 24"
-                  >
-                    <circle
-                      class="opacity-25"
-                      cx="12"
-                      cy="12"
-                      r="10"
-                      stroke="currentColor"
-                      stroke-width="4"
-                    />
-                    <path
-                      class="opacity-75"
-                      fill="currentColor"
-                      d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z"
-                    />
-                  </svg>
-                  {{ isCreatingInvoice ? 'Creating...' : invoiceButtonText }}
-                </button>
-
-                <p v-if="props.paid" class="text-xs text-slate-500 text-center">
-                  Job is marked as <strong>Paid</strong>. Unmark "Paid" to create another invoice.
-                </p>
-              </CardFooter>
-            </div>
-          </Card>
-        </div>
       </aside>
     </div>
 
@@ -268,174 +128,6 @@
         </DialogFooter>
       </DialogContent>
     </Dialog>
-
-    <!-- Create Invoice Modal -->
-    <Dialog :open="showInvoiceModal" @update:open="showInvoiceModal = $event">
-      <DialogContent class="sm:max-w-md">
-        <DialogHeader>
-          <DialogTitle>Create Invoice</DialogTitle>
-          <DialogDescription>
-            Choose how to invoice
-            {{
-              props.pricingMethodology === 'fixed_price'
-                ? 'this quoted job'
-                : 'time &amp; materials'
-            }}
-          </DialogDescription>
-        </DialogHeader>
-        <div class="space-y-4 py-4">
-          <!-- Calculation Context -->
-          <div class="rounded-lg bg-slate-50 p-3 text-sm space-y-1">
-            <div class="flex justify-between" v-if="props.pricingMethodology === 'fixed_price'">
-              <span class="text-slate-600">Quote total</span>
-              <span class="font-semibold tabular-nums">{{ formatCurrency(quoteTotal) }}</span>
-            </div>
-            <div class="flex justify-between" v-else>
-              <span class="text-slate-600">Actual revenue to date</span>
-              <span class="font-semibold tabular-nums">{{
-                formatCurrency(actualSummary.rev)
-              }}</span>
-            </div>
-            <div class="flex justify-between">
-              <span class="text-slate-600">Already invoiced</span>
-              <span class="font-semibold tabular-nums">{{ formatCurrency(invoiceTotal) }}</span>
-            </div>
-            <div class="flex justify-between border-t pt-1">
-              <span class="text-slate-700 font-medium">Current amount to invoice</span>
-              <span class="font-bold text-orange-600 tabular-nums">{{
-                formatCurrency(toBeInvoiced)
-              }}</span>
-            </div>
-          </div>
-
-          <!-- Mode Options -->
-          <div class="space-y-3">
-            <!-- Quoted job options -->
-            <template v-if="props.pricingMethodology === 'fixed_price'">
-              <Button
-                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
-                variant="default"
-                @click="executeCreateInvoice('invoice_full')"
-                :disabled="isCreatingInvoice"
-              >
-                <span class="font-semibold">Invoice remaining quote</span>
-                <span class="text-xs font-normal opacity-90">
-                  Invoice the quote balance: {{ formatCurrency(toBeInvoiced) }}
-                </span>
-              </Button>
-
-              <Button
-                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
-                variant="outline"
-                @click="selectInvoiceMode('invoice_percent')"
-                :disabled="isCreatingInvoice"
-              >
-                <span class="font-semibold">Invoice % of quote</span>
-                <span class="text-xs font-normal opacity-70">
-                  Invoice a cumulative percentage of the quoted amount
-                </span>
-              </Button>
-              <div v-if="selectedInvoiceMode === 'invoice_percent'" class="flex gap-2">
-                <Input
-                  v-model="invoicePercentInput"
-                  type="number"
-                  min="1"
-                  max="100"
-                  step="0.1"
-                  placeholder="e.g. 50"
-                  @keydown.enter.prevent="executeCreateInvoice('invoice_percent')"
-                />
-                <Button
-                  @click="executeCreateInvoice('invoice_percent')"
-                  size="sm"
-                  :disabled="isCreatingInvoice"
-                >
-                  Confirm
-                </Button>
-              </div>
-
-              <Button
-                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
-                variant="outline"
-                @click="selectInvoiceMode('invoice_amount')"
-                :disabled="isCreatingInvoice"
-              >
-                <span class="font-semibold">Invoice $ amount</span>
-                <span class="text-xs font-normal opacity-70">
-                  Invoice a specific dollar amount (must not exceed remaining)
-                </span>
-              </Button>
-              <div v-if="selectedInvoiceMode === 'invoice_amount'" class="flex gap-2">
-                <Input
-                  v-model="invoiceAmountInput"
-                  type="number"
-                  min="0"
-                  step="0.01"
-                  placeholder="0.00"
-                  @keydown.enter.prevent="executeCreateInvoice('invoice_amount')"
-                />
-                <Button
-                  @click="executeCreateInvoice('invoice_amount')"
-                  size="sm"
-                  :disabled="isCreatingInvoice"
-                >
-                  Confirm
-                </Button>
-              </div>
-            </template>
-
-            <!-- T&M job options -->
-            <template v-else>
-              <Button
-                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
-                variant="default"
-                @click="executeCreateInvoice('invoice_costs_to_date')"
-                :disabled="isCreatingInvoice"
-              >
-                <span class="font-semibold">Invoice costs to date</span>
-                <span class="text-xs font-normal opacity-90">
-                  Invoice actual revenue minus already invoiced: {{ formatCurrency(toBeInvoiced) }}
-                </span>
-              </Button>
-
-              <Button
-                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
-                variant="outline"
-                @click="selectInvoiceMode('invoice_amount')"
-                :disabled="isCreatingInvoice"
-              >
-                <span class="font-semibold">Invoice $ amount</span>
-                <span class="text-xs font-normal opacity-70">
-                  Invoice a specific dollar amount (progress/material payment)
-                </span>
-              </Button>
-              <div v-if="selectedInvoiceMode === 'invoice_amount'" class="flex gap-2">
-                <Input
-                  v-model="invoiceAmountInput"
-                  type="number"
-                  min="0"
-                  step="0.01"
-                  placeholder="0.00"
-                  @keydown.enter.prevent="executeCreateInvoice('invoice_amount')"
-                />
-                <Button
-                  @click="executeCreateInvoice('invoice_amount')"
-                  size="sm"
-                  :disabled="isCreatingInvoice"
-                >
-                  Confirm
-                </Button>
-              </div>
-            </template>
-          </div>
-        </div>
-        <DialogFooter>
-          <Button variant="ghost" @click="showInvoiceModal = false" :disabled="isCreatingInvoice"
-            >Cancel</Button
-          >
-        </DialogFooter>
-      </DialogContent>
-    </Dialog>
   </div>
 </template>
 
@@ -444,7 +136,7 @@ import debug from 'debug'
 
 const log = debug('job:actual')
 import { toLocalDateString } from '../../utils/dateUtils'
-import { formatCurrency, formatDate } from '@/utils/string-formatting'
+import { formatCurrency } from '@/utils/string-formatting'
 import { normalizeOptionalDecimal } from '@/utils/number'
 import { onMounted, ref, computed, reactive } from 'vue'
 import { useRouter } from 'vue-router'
@@ -457,10 +149,8 @@ import { schemas } from '../../api/generated/api'
 import { useSmartCostLineDelete } from '../../composables/useSmartCostLineDelete'
 import { useCostSummary } from '../../composables/useCostSummary'
 import { useCostLineDrafts } from '@/composables/useCostLineDrafts'
-import { useXeroConnection } from '../../composables/useXeroConnection'
 import { api } from '../../api/client'
 import { z } from 'zod'
-import type { AxiosError } from 'axios'
 import type { KindOption } from '../shared/SmartCostLinesTable.vue'
 import { useStockStore } from '../../stores/stockStore'
 import SmartCostLinesTable from '../shared/SmartCostLinesTable.vue'
@@ -473,18 +163,12 @@ import {
   DialogFooter,
 } from '../ui/dialog'
 import { Button } from '../ui/button'
-import { Card, CardHeader, CardFooter, CardContent, CardDescription, CardTitle } from '../ui/card'
-import { Input } from '../ui/input'
-import { ExternalLink, Trash2 } from 'lucide-vue-next'
-import { Badge } from '../ui/badge'
 import { useSaveFeedback } from '@/composables/useSaveFeedback'
 
 type CostLine = z.infer<typeof schemas.CostLine>
 type CostSet = z.infer<typeof schemas.CostSet>
 type KanbanStaff = z.infer<typeof schemas.KanbanStaff>
 type StockConsumeRequest = z.infer<typeof schemas.StockConsumeRequest>
-type Invoice = z.infer<typeof schemas.Invoice>
-type XeroInvoiceCreateRequest = z.infer<typeof schemas.XeroInvoiceCreateRequest>
 
 // Type guard functions to safely access meta and ext_refs
 function isDeliveryReceiptMeta(meta: unknown): meta is { source: string; po_number?: string } {
@@ -526,11 +210,7 @@ function isStockExtRefs(extRefs: unknown): extRefs is { stock_id: string } {
 
 const props = defineProps<{
   jobId: string
-  jobNumber: string
   pricingMethodology: string
-  quoted: boolean
-  fullyInvoiced: boolean
-  paid?: boolean
 }>()
 const jobActualSaveFeedback = useSaveFeedback(`job-actual:${props.jobId}`, {
   toastErrors: false,
@@ -538,162 +218,17 @@ const jobActualSaveFeedback = useSaveFeedback(`job-actual:${props.jobId}`, {
 
 const emit = defineEmits<{
   'cost-line-changed': []
-  'quote-created': []
-  'quote-accepted': []
-  'invoice-created': []
-  'quote-deleted': []
-  'invoice-deleted': []
 }>()
 
 const stockStore = useStockStore()
 
-// Local state for invoices
-const isCreatingInvoice = ref(false)
-const deletingInvoiceId = ref<string | null>(null) // Track which invoice is being deleted
-const { xeroConnected } = useXeroConnection()
-
-// Local state for KPIs
+// KPI chips. Invoice figures deliberately live in Finish Job only, so there is
+// one authoritative place showing what the customer owes.
 const estimateTotal = ref(0)
 const quoteTotal = ref(0)
 const costsSummaryLoading = ref(false)
-
-// --- COMPUTED PROPERTIES ---
-const invoices = ref<Array<Invoice>>([])
 const timeAndExpenses = computed(() => actualSummary.value.rev)
 
-const invoiceTotal = computed(() => {
-  if (!invoices.value.length) return 0
-  return invoices.value
-    .filter((inv) => inv.status !== 'VOIDED' && inv.status !== 'DELETED')
-    .reduce((sum, inv) => sum + (inv.total_excl_tax || 0), 0)
-})
-
-const toBeInvoiced = computed(() => {
-  // For fixed price jobs with a quote, use the quoted amount
-  // For T&M jobs or jobs without quotes, use the actual time & expenses
-  const amountToInvoice =
-    props.pricingMethodology === 'fixed_price' && quoteTotal.value > 0
-      ? quoteTotal.value
-      : actualSummary.value.rev
-
-  return amountToInvoice - invoiceTotal.value
-})
-
-const invoiceButtonText = computed(() => {
-  if (!xeroConnected.value) {
-    return 'Login to Xero first'
-  }
-  return 'Create Invoice'
-})
-
-// --- INVOICE MODAL ---
-const showInvoiceModal = ref(false)
-const selectedInvoiceMode = ref<string | null>(null)
-const invoicePercentInput = ref('')
-const invoiceAmountInput = ref('')
-
-const createInvoice = () => {
-  selectedInvoiceMode.value = null
-  invoicePercentInput.value = ''
-  invoiceAmountInput.value = ''
-  showInvoiceModal.value = true
-}
-
-function selectInvoiceMode(mode: string) {
-  selectedInvoiceMode.value = mode
-  invoicePercentInput.value = ''
-  invoiceAmountInput.value = ''
-}
-
-async function executeCreateInvoice(mode: string) {
-  if (!props.jobId || isCreatingInvoice.value) return
-
-  let percent: number | undefined
-  let amount: number | undefined
-
-  if (mode === 'invoice_percent') {
-    const pct = parseFloat(invoicePercentInput.value)
-    if (isNaN(pct) || pct <= 0 || pct > 100) {
-      toast.error('Please enter a valid percentage (1-100)')
-      return
-    }
-    percent = pct
-  }
-
-  if (mode === 'invoice_amount') {
-    const amt = parseFloat(invoiceAmountInput.value)
-    if (isNaN(amt) || amt <= 0) {
-      toast.error('Please enter a valid amount')
-      return
-    }
-    amount = amt
-  }
-
-  showInvoiceModal.value = false
-  isCreatingInvoice.value = true
-
-  try {
-    const body = { mode } as XeroInvoiceCreateRequest
-    if (percent !== undefined) body.percent = percent
-    if (amount !== undefined) body.amount = amount
-
-    const response = await api.xero_create_invoice_create(body, {
-      params: { job_id: props.jobId },
-    })
-    if (!response?.success) {
-      const msgs = response?.messages?.length ? response.messages : ['Failed to create invoice']
-      msgs.forEach((msg: string) => toast.error(msg))
-      return
-    }
-    toast.success('Invoice created successfully!')
-    if (response.messages?.length) {
-      response.messages.forEach((msg: string) => toast.warning(msg))
-    }
-    emit('invoice-created')
-    await loadInvoices()
-  } catch (err: unknown) {
-    let msg = 'Unexpected error while trying to create invoice.'
-    log('Error creating invoice:', err)
-    if ((err as AxiosError).isAxiosError) {
-      const axiosErr = err as AxiosError<{ message: string }>
-      const errorData = axiosErr.response?.data
-      if (typeof errorData === 'object' && errorData !== null && 'error' in errorData) {
-        msg = String((errorData as { error: string }).error)
-      }
-    }
-    toast.error(`Failed to create invoice: ${msg}`)
-  } finally {
-    isCreatingInvoice.value = false
-  }
-}
-
-const goToInvoiceOnXero = (url: string | null | undefined) => {
-  if (url && url !== '#') {
-    window.open(url, '_blank')
-  } else {
-    toast.error('No online URL available for this invoice.')
-  }
-}
-
-const deleteInvoiceOnXero = async (invoiceXeroId: string) => {
-  if (!props.jobId || deletingInvoiceId.value) return
-  deletingInvoiceId.value = invoiceXeroId
-  try {
-    await api.xero_delete_invoice_destroy(undefined, {
-      params: { job_id: props.jobId },
-      queries: { xero_invoice_id: invoiceXeroId },
-    })
-    toast.success('Invoice deleted successfully!')
-    emit('invoice-deleted')
-    await loadInvoices() // Reload invoices after deletion
-  } catch (err) {
-    console.error('Error deleting invoice:', err)
-    toast.error('Failed to delete invoice.')
-  } finally {
-    deletingInvoiceId.value = null
-  }
-}
-
 const router = useRouter()
 
 const costLines = ref<CostLine[]>([])
@@ -777,18 +312,6 @@ async function loadCostsSummary() {
   }
 }
 
-async function loadInvoices() {
-  try {
-    const response = await api.job_jobs_invoices_retrieve({
-      params: { job_id: props.jobId },
-    })
-    // Zodios returns data directly, not wrapped in {success, data}
-    invoices.value = response.invoices || []
-  } catch (error) {
-    log('Failed to load invoices:', error)
-  }
-}
-
 // Use the smart delete composable
 const { handleSmartDelete } = useSmartCostLineDelete({
   costLines,
@@ -899,7 +422,7 @@ async function handleCreateLine(line: CostLine): Promise<CostLine> {
 const costLineDraftSession = useCostLineDrafts({ costLines, createLine: handleCreateLine })
 
 onMounted(async () => {
-  await Promise.all([loadStaff(), loadActualCosts(), loadCostsSummary(), loadInvoices()])
+  await Promise.all([loadStaff(), loadActualCosts(), loadCostsSummary()])
 })
 
 // Use the cost summary composable (simple version for actual)
diff --git a/frontend/src/components/job/JobCostAnalysisTab.vue b/frontend/src/components/job/JobFinishTab.vue
similarity index 51%
rename from frontend/src/components/job/JobCostAnalysisTab.vue
rename to frontend/src/components/job/JobFinishTab.vue
index f8270976c..6dcd10913 100644
--- a/frontend/src/components/job/JobCostAnalysisTab.vue
+++ b/frontend/src/components/job/JobFinishTab.vue
@@ -2,7 +2,7 @@
   <div class="space-y-6">
     <div class="flex items-center justify-between bg-white p-4 rounded-lg border border-gray-200">
       <div>
-        <h2 class="text-lg font-semibold text-gray-900">Cost Analysis</h2>
+        <h2 class="text-lg font-semibold text-gray-900">Finish Job</h2>
         <p class="text-sm text-gray-500">
           <span v-if="props.pricingMethodology === 'time_materials'">
             Compare Estimate and Actual cost sets for this Time & Materials job
@@ -22,11 +22,174 @@
         <div
           class="animate-spin rounded-full h-8 w-8 border-b-2 border-blue-600 mx-auto mb-2"
         ></div>
-        <p class="text-gray-500">Loading cost data...</p>
+        <p class="text-gray-500">Loading job financials...</p>
       </div>
     </div>
 
-    <div v-else class="overflow-x-auto">
+    <div
+      v-else-if="loadError"
+      data-automation-id="JobFinishTab-load-error"
+      class="bg-white rounded-lg border border-red-200 p-6 text-center"
+    >
+      <p class="text-red-700 font-medium">Could not load this job's financials.</p>
+      <p class="text-sm text-slate-600 mt-1">
+        Reload the page. Do not quote the customer a figure until it loads.
+      </p>
+    </div>
+
+    <template v-else-if="summary">
+      <div class="grid grid-cols-1 lg:grid-cols-[1fr_360px] gap-4 items-start">
+        <section
+          data-automation-id="JobFinishTab-balance"
+          class="bg-white rounded-xl border border-slate-200 p-4"
+        >
+          <h3 class="text-base font-semibold text-gray-900">Customer balance</h3>
+
+          <dl class="space-y-1.5 text-sm mt-3">
+            <div class="flex justify-between">
+              <dt class="text-slate-600">{{ jobValueLabel }}</dt>
+              <dd class="tabular-nums font-medium">
+                {{ formatCurrency(summary.job_value_excl_gst) }}
+              </dd>
+            </div>
+            <div class="flex justify-between">
+              <dt class="text-slate-600">Already invoiced (excl GST)</dt>
+              <dd class="tabular-nums font-medium">
+                {{ formatCurrency(summary.valid_invoiced_excl_gst) }}
+              </dd>
+            </div>
+            <div class="border-t border-slate-200 pt-1.5 flex justify-between">
+              <dt class="text-slate-600">Still to invoice (excl GST)</dt>
+              <dd
+                data-automation-id="JobFinishTab-remaining-excl-gst"
+                class="tabular-nums font-medium"
+              >
+                {{ formatCurrency(summary.remaining_to_invoice_excl_gst) }}
+              </dd>
+            </div>
+            <div class="flex justify-between">
+              <dt class="text-slate-600">GST</dt>
+              <dd data-automation-id="JobFinishTab-remaining-gst" class="tabular-nums font-medium">
+                {{ formatCurrency(summary.remaining_gst) }}
+              </dd>
+            </div>
+            <div v-if="summary.outstanding_invoiced_incl_gst > 0" class="flex justify-between">
+              <dt class="text-slate-600">Unpaid invoices (incl GST)</dt>
+              <dd
+                data-automation-id="JobFinishTab-outstanding-incl-gst"
+                class="tabular-nums font-medium"
+              >
+                {{ formatCurrency(summary.outstanding_invoiced_incl_gst) }}
+              </dd>
+            </div>
+            <div class="border-t-2 border-slate-300 pt-2 mt-1 flex justify-between items-baseline">
+              <dt class="font-semibold text-slate-900">Total to pay</dt>
+              <dd
+                data-automation-id="JobFinishTab-total-to-pay"
+                class="tabular-nums text-2xl font-bold text-emerald-700"
+              >
+                {{ formatCurrency(summary.total_to_pay_incl_gst) }}
+              </dd>
+            </div>
+          </dl>
+
+          <div
+            v-if="summary.over_invoiced_excl_gst > 0"
+            data-automation-id="JobFinishTab-over-invoiced"
+            class="mt-3 rounded-md bg-amber-50 border border-amber-200 p-3 text-sm text-amber-900"
+          >
+            <strong>Over-invoiced by {{ formatCurrency(summary.over_invoiced_excl_gst) }}</strong>
+            (excl GST). Resolve it with a credit note in Xero.
+          </div>
+        </section>
+
+        <JobInvoiceCard
+          :job-id="props.jobId"
+          :pricing-methodology="props.pricingMethodology"
+          :remaining-to-invoice="summary.remaining_to_invoice_excl_gst"
+          :job-status="props.jobStatus"
+          :paid="props.paid"
+          @invoices-changed="reloadFinishSummary"
+        />
+      </div>
+
+      <section
+        data-automation-id="JobFinishTab-checklist"
+        class="bg-white rounded-xl border border-slate-200 p-4"
+      >
+        <h3 class="text-base font-semibold text-gray-900">Completion checklist</h3>
+        <p class="text-xs text-slate-500 mb-3">Recorded against the job. Never blocks invoicing.</p>
+
+        <div class="space-y-2">
+          <label
+            v-for="item in checklistItems"
+            :key="item.key"
+            class="flex items-start gap-2 text-sm cursor-pointer"
+          >
+            <input
+              v-model="checklist[item.key]"
+              type="checkbox"
+              :data-automation-id="`JobFinishTab-checklist-${item.key}`"
+              class="mt-0.5 h-4 w-4 rounded border-slate-300"
+              :disabled="savingChecklistKey !== null"
+              @change="onChecklistToggle(item.key)"
+            />
+            <span class="text-slate-800">{{ item.label }}</span>
+          </label>
+        </div>
+
+        <p
+          v-if="props.pricingMethodology === 'time_materials'"
+          data-automation-id="JobFinishTab-tm-urgency"
+          class="mt-3 text-sm text-slate-600"
+        >
+          On a Time &amp; Materials job the time and materials are what the customer pays. Get them
+          right before invoicing.
+        </p>
+      </section>
+    </template>
+
+    <div
+      v-if="summary && !loadError"
+      data-automation-id="JobFinishTab-labour-hours"
+      class="grid grid-cols-1 sm:grid-cols-3 gap-3 text-sm"
+    >
+      <div class="rounded-lg border border-slate-200 bg-white p-3">
+        <div class="text-[11px] uppercase tracking-wide text-slate-500">Labour budget</div>
+        <div
+          data-automation-id="JobFinishTab-labour-budget"
+          class="text-lg font-semibold tabular-nums text-slate-900"
+        >
+          {{ formatNumber(labourBudgetHours) }}
+        </div>
+      </div>
+      <div class="rounded-lg border border-slate-200 bg-white p-3">
+        <div class="text-[11px] uppercase tracking-wide text-slate-500">Hours used</div>
+        <div
+          data-automation-id="JobFinishTab-labour-used"
+          class="text-lg font-semibold tabular-nums text-slate-900"
+        >
+          {{ formatNumber(actual.hours) }}
+        </div>
+      </div>
+      <div
+        class="rounded-lg border bg-white p-3"
+        :class="labourOverrunHours > 0 ? 'border-red-200 bg-red-50' : 'border-slate-200'"
+      >
+        <div class="text-[11px] uppercase tracking-wide text-slate-500">
+          {{ labourOverrunHours > 0 ? 'Overrun' : 'Hours remaining' }}
+        </div>
+        <div
+          data-automation-id="JobFinishTab-labour-remaining"
+          class="text-lg font-semibold tabular-nums"
+          :class="labourOverrunHours > 0 ? 'text-red-700' : 'text-slate-900'"
+        >
+          {{ formatNumber(labourOverrunHours > 0 ? labourOverrunHours : labourRemainingHours) }}
+        </div>
+      </div>
+    </div>
+
+    <div v-if="summary && !loadError" class="overflow-x-auto">
       <table class="min-w-full bg-white rounded-lg border border-gray-200 text-sm">
         <thead>
           <tr class="bg-gray-50">
@@ -157,6 +320,8 @@ import { computed, ref, onMounted, reactive } from 'vue'
 import { toast } from 'vue-sonner'
 import { ArrowUp, ArrowDown, CheckCircle, AlertTriangle, XCircle } from 'lucide-vue-next'
 import { api } from '../../api/client'
+import debug from 'debug'
+import JobInvoiceCard from './JobInvoiceCard.vue'
 import { formatCurrency } from '@/utils/string-formatting'
 import { schemas } from '../../api/generated/api'
 import { z } from 'zod'
@@ -177,6 +342,8 @@ type JobCostSummaryResponse = z.infer<typeof schemas.JobCostSummaryResponse>
 const props = defineProps<{
   jobId: string
   pricingMethodology: string
+  jobStatus?: string
+  paid?: boolean
 }>()
 
 const loading = ref(true)
@@ -189,6 +356,39 @@ const zeroSummary = (): NormalizedCostSummary => ({
   profitMargin: 0,
 })
 
+const log = debug('job:finish')
+
+type FinishJobSummary = z.infer<typeof schemas.FinishJobSummary>
+type JobCompletionChecklist = z.infer<typeof schemas.JobCompletionChecklist>
+type ChecklistItemKey = keyof JobCompletionChecklist
+
+// No fabricated placeholder: until the server answers there is no balance, and a
+// failed load must not render as a settled $0 job.
+const summary = ref<FinishJobSummary | null>(null)
+const loadError = ref(false)
+const checklist = reactive<JobCompletionChecklist>({
+  foreman_signed_off: false,
+  timesheets_collected: false,
+  materials_checked: false,
+  customer_called: false,
+  released: false,
+})
+const savingChecklistKey = ref<ChecklistItemKey | null>(null)
+
+const checklistItems: Array<{ key: ChecklistItemKey; label: string }> = [
+  { key: 'foreman_signed_off', label: 'Has the foreman signed off the job?' },
+  { key: 'timesheets_collected', label: 'Have you collected the timesheet entries?' },
+  { key: 'materials_checked', label: 'Have you checked the materials on the job?' },
+  { key: 'customer_called', label: 'Have you called the customer?' },
+  { key: 'released', label: 'Has the job been released?' },
+]
+
+const jobValueLabel = computed(() =>
+  props.pricingMethodology === 'fixed_price'
+    ? 'Quote total (excl GST)'
+    : 'Job value to date (excl GST)',
+)
+
 const estimate = reactive<NormalizedCostSummary>(zeroSummary())
 const quote = reactive<NormalizedCostSummary>(zeroSummary())
 const actual = reactive<NormalizedCostSummary>(zeroSummary())
@@ -200,26 +400,37 @@ const ensureSummary = (s?: JobCostSetSummaryOutput | null): NormalizedCostSummar
   profitMargin: s?.profitMargin ?? 0,
 })
 
+async function loadFinishSummary() {
+  const response = await api.job_jobs_finish_retrieve({ params: { job_id: props.jobId } })
+  summary.value = response.summary
+  Object.assign(checklist, response.checklist)
+}
+
+async function loadCostSummary() {
+  const resp: JobCostSummaryResponse = await api.job_jobs_costs_summary_retrieve({
+    params: { job_id: props.jobId },
+  })
+
+  Object.assign(estimate, ensureSummary(resp.estimate))
+  Object.assign(quote, ensureSummary(resp.quote))
+  Object.assign(actual, ensureSummary(resp.actual))
+
+  hasValidQuoteData.value =
+    !!resp.quote && ((resp.quote.cost ?? 0) > 0 || (resp.quote.rev ?? 0) > 0)
+}
+
 async function loadAll() {
   loading.value = true
+  loadError.value = false
   try {
-    const resp: JobCostSummaryResponse = await api.job_jobs_costs_summary_retrieve({
-      params: { job_id: props.jobId },
-    })
-
-    Object.assign(estimate, ensureSummary(resp.estimate))
-    Object.assign(quote, ensureSummary(resp.quote))
-    Object.assign(actual, ensureSummary(resp.actual))
-
-    hasValidQuoteData.value =
-      !!resp.quote && ((resp.quote.cost ?? 0) > 0 || (resp.quote.rev ?? 0) > 0)
+    await Promise.all([loadFinishSummary(), loadCostSummary()])
   } catch (error) {
-    console.error('Failed to load cost summary:', error)
-    toast.error('Failed to load cost analysis')
-    Object.assign(estimate, zeroSummary())
-    Object.assign(quote, zeroSummary())
-    Object.assign(actual, zeroSummary())
-    hasValidQuoteData.value = false
+    log('Failed to load Finish Job data: %o', error)
+    toast.error('Failed to load job financials')
+    // Leave summary null and say so. Showing zeros here would tell staff a job
+    // is settled when we simply could not read it.
+    summary.value = null
+    loadError.value = true
   } finally {
     loading.value = false
   }
@@ -227,6 +438,49 @@ async function loadAll() {
 
 onMounted(loadAll)
 
+// Invoicing changed the balance, so a failed reload must say so rather than
+// leave the pre-invoice figure on screen looking current.
+async function reloadFinishSummary() {
+  try {
+    await loadFinishSummary()
+  } catch (error) {
+    log('Failed to reload Finish Job balance: %o', error)
+    summary.value = null
+    loadError.value = true
+    toast.error('Invoice saved, but the balance could not be refreshed. Reload the job.')
+  }
+}
+
+// v-model has already applied the new value optimistically; this persists it and
+// puts it back if the server refuses, so a tick never survives a failed save.
+async function onChecklistToggle(key: ChecklistItemKey) {
+  const checked = checklist[key]
+  savingChecklistKey.value = key
+  try {
+    const response = await api.job_jobs_finish_partial_update(
+      { [key]: checked },
+      { params: { job_id: props.jobId } },
+    )
+    Object.assign(checklist, response.checklist)
+  } catch (error) {
+    log('Failed to save checklist item %s: %o', key, error)
+    toast.error('Failed to save checklist')
+    checklist[key] = !checked
+  } finally {
+    savingChecklistKey.value = null
+  }
+}
+
+// --- KAN-222 labour hours ---
+
+const labourBudgetHours = computed(() =>
+  props.pricingMethodology === 'fixed_price' && hasValidQuoteData.value
+    ? quote.hours
+    : estimate.hours,
+)
+const labourRemainingHours = computed(() => Math.max(labourBudgetHours.value - actual.hours, 0))
+const labourOverrunHours = computed(() => Math.max(actual.hours - labourBudgetHours.value, 0))
+
 const showQuoteColumn = computed(() => {
   if (props.pricingMethodology === 'time_materials') return false
   return hasValidQuoteData.value || quote.cost > 0 || quote.rev > 0
diff --git a/frontend/src/components/job/JobInvoiceCard.vue b/frontend/src/components/job/JobInvoiceCard.vue
new file mode 100644
index 000000000..3023d3e08
--- /dev/null
+++ b/frontend/src/components/job/JobInvoiceCard.vue
@@ -0,0 +1,469 @@
+<template>
+  <aside class="bg-white rounded-xl border border-slate-200">
+    <Card class="border-0 shadow-none overflow-hidden">
+      <CardHeader class="px-3 pt-3 pb-2">
+        <CardTitle>
+          Invoices <span class="text-slate-400 text-sm">({{ invoices.length }})</span>
+        </CardTitle>
+        <CardDescription>Manage invoices for this job.</CardDescription>
+      </CardHeader>
+
+      <CardContent class="p-0 pb-2">
+        <div class="max-h-[20rem] overflow-y-auto px-2" style="scrollbar-gutter: stable">
+          <div v-if="invoices.length === 0" class="text-center py-6 text-gray-500">
+            No invoices for this project
+          </div>
+
+          <ul
+            v-else
+            role="list"
+            data-automation-id="JobInvoiceCard-list"
+            class="divide-y divide-slate-200 rounded-md bg-white"
+          >
+            <li
+              v-for="invoice in invoices"
+              :key="invoice.id"
+              class="px-3 py-1.5 hover:bg-slate-50 flex items-center gap-3"
+            >
+              <div class="min-w-0 flex-1">
+                <div class="flex items-center gap-2">
+                  <span class="font-medium text-slate-900 text-sm leading-5">
+                    {{ invoice.number }}
+                  </span>
+                  <Badge
+                    :variant="invoice.status === 'PAID' ? 'default' : 'secondary'"
+                    class="text-[10px] px-1.5 py-0.5 rounded-full"
+                  >
+                    {{ invoice.status }}
+                  </Badge>
+                </div>
+                <div class="text-[11px] leading-4 text-slate-500">
+                  {{ formatDate(invoice.date) }}
+                </div>
+              </div>
+
+              <div class="shrink-0 text-sm font-semibold tabular-nums text-slate-900">
+                {{ formatCurrency(invoice.total_excl_tax) }}
+              </div>
+
+              <div class="shrink-0 flex items-center gap-2">
+                <Button
+                  variant="outline"
+                  size="icon"
+                  class="h-7 w-7"
+                  :data-automation-id="`JobInvoiceCard-open-${invoice.id}`"
+                  :aria-label="`Open invoice ${invoice.number} in Xero`"
+                  @click="goToInvoiceOnXero(invoice.online_url)"
+                  :disabled="!invoice.online_url"
+                >
+                  <ExternalLink class="h-4 w-4" />
+                </Button>
+                <Button
+                  variant="destructive"
+                  size="icon"
+                  class="h-7 w-7"
+                  :data-automation-id="`JobInvoiceCard-delete-${invoice.id}`"
+                  :aria-label="`Delete invoice ${invoice.number}`"
+                  @click="deleteInvoiceOnXero(invoice.xero_id)"
+                  :disabled="!!deletingInvoiceId"
+                >
+                  <svg
+                    v-if="deletingInvoiceId === invoice.xero_id"
+                    class="animate-spin h-4 w-4"
+                    viewBox="0 0 24 24"
+                  >
+                    <circle
+                      class="opacity-25"
+                      cx="12"
+                      cy="12"
+                      r="10"
+                      stroke="currentColor"
+                      stroke-width="4"
+                    />
+                    <path
+                      class="opacity-75"
+                      fill="currentColor"
+                      d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z"
+                    />
+                  </svg>
+                  <Trash2 v-else class="h-4 w-4" />
+                </Button>
+              </div>
+            </li>
+          </ul>
+        </div>
+      </CardContent>
+
+      <div>
+        <div class="border-t border-slate-200"></div>
+        <CardFooter class="flex flex-col items-center gap-2 pt-4">
+          <button
+            v-if="props.remainingToInvoice > 0"
+            data-automation-id="JobFinishTab-create-invoice"
+            @click="openInvoiceModal()"
+            :disabled="isCreatingInvoice || !!props.paid || !xeroConnected"
+            class="px-4 py-2 bg-orange-600 text-white rounded-md hover:bg-orange-700 focus:outline-none focus:ring-2 focus:ring-orange-500 disabled:opacity-50 flex items-center gap-2"
+          >
+            <svg
+              v-if="isCreatingInvoice"
+              class="animate-spin -ml-1 mr-1 h-4 w-4"
+              viewBox="0 0 24 24"
+            >
+              <circle
+                class="opacity-25"
+                cx="12"
+                cy="12"
+                r="10"
+                stroke="currentColor"
+                stroke-width="4"
+              />
+              <path
+                class="opacity-75"
+                fill="currentColor"
+                d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z"
+              />
+            </svg>
+            {{ isCreatingInvoice ? 'Creating...' : invoiceButtonText }}
+          </button>
+
+          <p
+            v-else
+            data-automation-id="JobFinishTab-fully-invoiced"
+            class="text-xs text-slate-500 text-center"
+          >
+            Fully invoiced — nothing further to invoice.
+          </p>
+
+          <p v-if="props.paid" class="text-xs text-slate-500 text-center">
+            Job is marked as <strong>Paid</strong>. Unmark "Paid" to create another invoice.
+          </p>
+        </CardFooter>
+      </div>
+    </Card>
+
+    <Dialog :open="showInvoiceModal" @update:open="showInvoiceModal = $event">
+      <DialogContent class="sm:max-w-md">
+        <DialogHeader>
+          <DialogTitle>Create Invoice</DialogTitle>
+          <DialogDescription>{{ invoiceModalDescription }}</DialogDescription>
+        </DialogHeader>
+        <div class="space-y-4 py-4">
+          <div class="rounded-lg bg-slate-50 p-3 text-sm space-y-1">
+            <div class="flex justify-between border-t pt-1">
+              <span class="text-slate-700 font-medium">Current amount to invoice</span>
+              <span class="font-bold text-orange-600 tabular-nums">
+                {{ formatCurrency(props.remainingToInvoice) }}
+              </span>
+            </div>
+          </div>
+
+          <div class="space-y-3">
+            <template v-if="props.pricingMethodology === 'fixed_price'">
+              <Button
+                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
+                variant="default"
+                data-automation-id="JobFinishTab-mode-invoice-full"
+                @click="executeCreateInvoice('invoice_full')"
+                :disabled="isCreatingInvoice"
+              >
+                <span class="font-semibold">{{ fullInvoiceLabel }}</span>
+                <span class="text-xs font-normal opacity-90">
+                  {{ formatCurrency(props.remainingToInvoice) }} excl GST
+                </span>
+              </Button>
+
+              <Button
+                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
+                variant="outline"
+                data-automation-id="JobFinishTab-mode-invoice-percent"
+                @click="selectInvoiceMode('invoice_percent')"
+                :disabled="isCreatingInvoice"
+              >
+                <span class="font-semibold">Invoice % of quote</span>
+                <span class="text-xs font-normal opacity-70">
+                  Invoice a cumulative percentage of the quoted amount
+                </span>
+              </Button>
+              <div v-if="selectedInvoiceMode === 'invoice_percent'" class="flex gap-2">
+                <Input
+                  v-model="invoicePercentInput"
+                  type="number"
+                  min="1"
+                  max="100"
+                  step="0.1"
+                  placeholder="e.g. 50"
+                  @keydown.enter.prevent="executeCreateInvoice('invoice_percent')"
+                />
+                <Button
+                  @click="executeCreateInvoice('invoice_percent')"
+                  size="sm"
+                  :disabled="isCreatingInvoice"
+                >
+                  Confirm
+                </Button>
+              </div>
+            </template>
+
+            <template v-else>
+              <Button
+                class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
+                variant="default"
+                data-automation-id="JobFinishTab-mode-invoice-costs-to-date"
+                @click="executeCreateInvoice('invoice_costs_to_date')"
+                :disabled="isCreatingInvoice"
+              >
+                <span class="font-semibold">{{ costsToDateLabel }}</span>
+                <span class="text-xs font-normal opacity-90">
+                  {{ formatCurrency(props.remainingToInvoice) }} excl GST
+                </span>
+              </Button>
+            </template>
+
+            <Button
+              class="w-full h-auto py-4 px-4 flex flex-col items-start gap-1"
+              variant="outline"
+              data-automation-id="JobFinishTab-mode-invoice-amount"
+              @click="selectInvoiceMode('invoice_amount')"
+              :disabled="isCreatingInvoice"
+            >
+              <span class="font-semibold">Invoice $ amount</span>
+              <span class="text-xs font-normal opacity-70">{{ customAmountHint }}</span>
+            </Button>
+            <div v-if="selectedInvoiceMode === 'invoice_amount'" class="flex gap-2">
+              <Input
+                v-model="invoiceAmountInput"
+                type="number"
+                min="0"
+                step="0.01"
+                placeholder="0.00"
+                @keydown.enter.prevent="executeCreateInvoice('invoice_amount')"
+              />
+              <Button
+                @click="executeCreateInvoice('invoice_amount')"
+                size="sm"
+                :disabled="isCreatingInvoice"
+              >
+                Confirm
+              </Button>
+            </div>
+          </div>
+        </div>
+        <DialogFooter>
+          <Button variant="ghost" @click="showInvoiceModal = false" :disabled="isCreatingInvoice">
+            Cancel
+          </Button>
+        </DialogFooter>
+      </DialogContent>
+    </Dialog>
+  </aside>
+</template>
+
+<script setup lang="ts">
+import { computed, onMounted, ref } from 'vue'
+import { toast } from 'vue-sonner'
+import debug from 'debug'
+import { ExternalLink, Trash2 } from 'lucide-vue-next'
+import { api } from '@/api/client'
+import { schemas } from '@/api/generated/api'
+import { formatCurrency, formatDate } from '@/utils/string-formatting'
+import { useXeroConnection } from '@/composables/useXeroConnection'
+import {
+  Dialog,
+  DialogContent,
+  DialogHeader,
+  DialogTitle,
+  DialogDescription,
+  DialogFooter,
+} from '../ui/dialog'
+import { Button } from '../ui/button'
+import { Card, CardHeader, CardFooter, CardContent, CardDescription, CardTitle } from '../ui/card'
+import { Input } from '../ui/input'
+import { Badge } from '../ui/badge'
+import { isAxiosError } from 'axios'
+import { z } from 'zod'
+
+const log = debug('job:invoice')
+
+type Invoice = z.infer<typeof schemas.Invoice>
+type XeroInvoiceCreateRequest = z.infer<typeof schemas.XeroInvoiceCreateRequest>
+type XeroInvoiceCreateMode = XeroInvoiceCreateRequest['mode']
+
+// Job statuses that mean the work itself is done. Purely a wording choice —
+// invoicing stays available at every status.
+const COMPLETED_STATUSES = ['recently_completed', 'archived']
+
+const props = defineProps<{
+  jobId: string
+  pricingMethodology: string
+  remainingToInvoice: number
+  jobStatus?: string
+  paid?: boolean
+}>()
+
+const emit = defineEmits<{ 'invoices-changed': [] }>()
+
+const invoices = ref<Array<Invoice>>([])
+const isCreatingInvoice = ref(false)
+const deletingInvoiceId = ref<string | null>(null)
+const { xeroConnected } = useXeroConnection()
+
+const showInvoiceModal = ref(false)
+const selectedInvoiceMode = ref<XeroInvoiceCreateMode | null>(null)
+const invoicePercentInput = ref('')
+const invoiceAmountInput = ref('')
+
+async function loadInvoices() {
+  const response = await api.job_jobs_invoices_retrieve({ params: { job_id: props.jobId } })
+  invoices.value = response.invoices || []
+}
+
+onMounted(async () => {
+  try {
+    await loadInvoices()
+  } catch (error) {
+    log('Failed to load invoices: %o', error)
+    toast.error('Failed to load invoices')
+  }
+})
+
+const invoiceButtonText = computed(() =>
+  xeroConnected.value ? 'Create Invoice' : 'Login to Xero first',
+)
+
+const workComplete = computed(() => COMPLETED_STATUSES.includes(props.jobStatus ?? ''))
+
+const invoiceModalDescription = computed(() => {
+  if (!workComplete.value) {
+    return 'Request payment in advance, take a deposit, or raise a progress invoice.'
+  }
+  return props.pricingMethodology === 'fixed_price'
+    ? 'Invoice the remaining quote balance.'
+    : 'Invoice the remaining Time & Materials costs.'
+})
+
+const fullInvoiceLabel = computed(() =>
+  workComplete.value ? 'Invoice remaining quote balance' : 'Invoice full amount in advance',
+)
+
+const costsToDateLabel = computed(() =>
+  workComplete.value ? 'Invoice remaining T&M costs' : 'Invoice costs to date',
+)
+
+const customAmountHint = computed(() =>
+  workComplete.value
+    ? 'Invoice a specific dollar amount'
+    : 'Invoice a deposit or progress payment of a specific amount',
+)
+
+function openInvoiceModal() {
+  selectedInvoiceMode.value = null
+  invoicePercentInput.value = ''
+  invoiceAmountInput.value = ''
+  showInvoiceModal.value = true
+}
+
+function selectInvoiceMode(mode: XeroInvoiceCreateMode) {
+  selectedInvoiceMode.value = mode
+  invoicePercentInput.value = ''
+  invoiceAmountInput.value = ''
+}
+
+async function executeCreateInvoice(mode: XeroInvoiceCreateMode) {
+  if (!props.jobId || isCreatingInvoice.value) return
+
+  let percent: number | undefined
+  let amount: number | undefined
+
+  if (mode === 'invoice_percent') {
+    const pct = parseFloat(invoicePercentInput.value)
+    if (isNaN(pct) || pct <= 0 || pct > 100) {
+      toast.error('Please enter a valid percentage (1-100)')
+      return
+    }
+    percent = pct
+  }
+
+  if (mode === 'invoice_amount') {
+    const amt = parseFloat(invoiceAmountInput.value)
+    if (isNaN(amt) || amt <= 0) {
+      toast.error('Please enter a valid amount')
+      return
+    }
+    amount = amt
+  }
+
+  showInvoiceModal.value = false
+  isCreatingInvoice.value = true
+
+  try {
+    const body: XeroInvoiceCreateRequest = {
+      mode,
+      ...(percent !== undefined && { percent }),
+      ...(amount !== undefined && { amount }),
+    }
+
+    const response = await api.xero_create_invoice_create(body, {
+      params: { job_id: props.jobId },
+    })
+    if (!response?.success) {
+      const msgs = response?.messages?.length ? response.messages : ['Failed to create invoice']
+      msgs.forEach((msg: string) => toast.error(msg))
+      return
+    }
+    toast.success('Invoice created successfully!')
+    if (response.messages?.length) {
+      response.messages.forEach((msg: string) => toast.warning(msg))
+    }
+    await afterInvoiceChange()
+  } catch (err: unknown) {
+    let msg = 'Unexpected error while trying to create invoice.'
+    log('Error creating invoice: %o', err)
+    // The payload is typed with the property this code actually reads, so the
+    // declared shape and the access below cannot drift apart.
+    if (isAxiosError<{ error?: string }>(err) && err.response?.data?.error) {
+      msg = err.response.data.error
+    }
+    toast.error(`Failed to create invoice: ${msg}`)
+  } finally {
+    isCreatingInvoice.value = false
+  }
+}
+
+const goToInvoiceOnXero = (url: string | null | undefined) => {
+  if (url && url !== '#') {
+    window.open(url, '_blank')
+  } else {
+    toast.error('No online URL available for this invoice.')
+  }
+}
+
+const deleteInvoiceOnXero = async (invoiceXeroId: string) => {
+  if (!props.jobId || deletingInvoiceId.value) return
+  deletingInvoiceId.value = invoiceXeroId
+  try {
+    await api.xero_delete_invoice_destroy(undefined, {
+      params: { job_id: props.jobId },
+      queries: { xero_invoice_id: invoiceXeroId },
+    })
+    toast.success('Invoice deleted successfully!')
+    await afterInvoiceChange()
+  } catch (err) {
+    log('Error deleting invoice: %o', err)
+    toast.error('Failed to delete invoice.')
+  } finally {
+    deletingInvoiceId.value = null
+  }
+}
+
+// The balance is only correct against the current invoice set, so the parent
+// re-reads it from the server rather than adjusting it locally.
+async function afterInvoiceChange() {
+  emit('invoices-changed')
+  try {
+    await loadInvoices()
+  } catch (error) {
+    log('Failed to reload invoices: %o', error)
+    toast.error('Invoice saved, but the list could not be refreshed. Reload the job.')
+  }
+}
+</script>
diff --git a/frontend/src/components/job/JobViewTabs.vue b/frontend/src/components/job/JobViewTabs.vue
index 203954c77..8e0daf174 100644
--- a/frontend/src/components/job/JobViewTabs.vue
+++ b/frontend/src/components/job/JobViewTabs.vue
@@ -65,21 +65,17 @@
       <div v-if="activeTab === 'actual'" class="h-full p-4 md:p-6">
         <JobActualTab
           :job-id="jobId"
-          :job-number="jobNumberString"
           :pricing-methodology="pricingMethodologyString"
-          :quoted="quotedBoolean"
-          :fully-invoiced="fullyInvoicedBoolean"
-          :paid="props.paid"
           @cost-line-changed="$emit('reload-job')"
-          @quote-created="$emit('quote-created')"
-          @quote-accepted="$emit('quote-accepted')"
-          @invoice-created="$emit('invoice-created')"
-          @quote-deleted="$emit('quote-deleted')"
-          @invoice-deleted="$emit('invoice-deleted')"
         />
       </div>
-      <div v-if="activeTab === 'costAnalysis'" class="h-full p-4 md:p-6">
-        <JobCostAnalysisTab :job-id="jobId" :pricing-methodology="pricingMethodologyString" />
+      <div v-if="activeTab === 'finishJob'" class="h-full p-4 md:p-6">
+        <JobFinishTab
+          :job-id="jobId"
+          :pricing-methodology="pricingMethodologyString"
+          :job-status="jobStatusString"
+          :paid="props.paid"
+        />
       </div>
       <div v-if="activeTab === 'jobSettings'" class="h-full p-4 md:p-6">
         <JobSettingsTab
@@ -119,7 +115,7 @@ const log = debug('job:view')
 import JobEstimateTab from './JobEstimateTab.vue'
 import JobQuoteTab from './JobQuoteTab.vue'
 import JobActualTab from './JobActualTab.vue'
-import JobCostAnalysisTab from './JobCostAnalysisTab.vue'
+import JobFinishTab from './JobFinishTab.vue'
 import JobSettingsTab from './JobSettingsTab.vue'
 import JobHistoryTab from './JobHistoryTab.vue'
 import JobAttachmentsTab from './JobAttachmentsTab.vue'
@@ -140,11 +136,6 @@ const emit = defineEmits<{
   (e: 'open-attachments'): void
   (e: 'open-pdf'): void
   (e: 'quote-imported', result: unknown): void
-  (e: 'quote-created'): void
-  (e: 'quote-accepted'): void
-  (e: 'invoice-created'): void
-  (e: 'quote-deleted'): void
-  (e: 'invoice-deleted'): void
   (e: 'delete-job'): void
   (e: 'reload-job'): void
   (e: 'job-updated', job: unknown): void
@@ -157,7 +148,7 @@ const allTabs: { key: JobTabKey; label: string }[] = [
   { key: 'estimate', label: 'Estimate' },
   { key: 'quote', label: 'Quote' },
   { key: 'actual', label: 'Actual' },
-  { key: 'costAnalysis', label: 'Cost Analysis' },
+  { key: 'finishJob', label: 'Finish Job' },
   { key: 'jobSettings', label: 'Job Settings' },
   { key: 'history', label: 'History' },
   { key: 'attachments', label: 'Attachments' },
diff --git a/frontend/src/components/job/__tests__/JobFinishTab.balance.test.ts b/frontend/src/components/job/__tests__/JobFinishTab.balance.test.ts
new file mode 100644
index 000000000..43ffeea70
--- /dev/null
+++ b/frontend/src/components/job/__tests__/JobFinishTab.balance.test.ts
@@ -0,0 +1,252 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { flushPromises } from '@vue/test-utils'
+
+const {
+  finishRetrieveMock,
+  invoicesRetrieveMock,
+  costsSummaryRetrieveMock,
+  createInvoiceMock,
+  deleteInvoiceMock,
+} = vi.hoisted(() => ({
+  finishRetrieveMock: vi.fn(),
+  invoicesRetrieveMock: vi.fn(),
+  costsSummaryRetrieveMock: vi.fn(),
+  createInvoiceMock: vi.fn(),
+  deleteInvoiceMock: vi.fn(),
+}))
+
+vi.mock('@/api/client', () => ({
+  api: {
+    job_jobs_finish_retrieve: finishRetrieveMock,
+    job_jobs_invoices_retrieve: invoicesRetrieveMock,
+    job_jobs_costs_summary_retrieve: costsSummaryRetrieveMock,
+    job_jobs_finish_partial_update: vi.fn(),
+    xero_create_invoice_create: createInvoiceMock,
+    xero_delete_invoice_destroy: deleteInvoiceMock,
+  },
+}))
+
+vi.mock('@/composables/useXeroConnection', () => ({
+  useXeroConnection: () => ({ xeroConnected: { value: true } }),
+}))
+
+vi.mock('vue-sonner', () => ({
+  toast: { error: vi.fn(), success: vi.fn(), warning: vi.fn() },
+}))
+
+import JobFinishTab from '../JobFinishTab.vue'
+import {
+  checklistState as checklist,
+  costSummary,
+  finishSummary as summary,
+  inModal,
+  modalText,
+  mountFinishTab,
+  resetFinishTab,
+} from './finishTabFixtures'
+
+/**
+ * The backend owns every figure here. These tests assert the component renders
+ * what the API returned and never recomputes it — the ADR 0020 boundary that
+ * KAN-323 exists to restore.
+ */
+const mountTab = (pricingMethodology = 'fixed_price', jobStatus = 'in_progress') =>
+  mountFinishTab(JobFinishTab, pricingMethodology, jobStatus)
+
+const text = (wrapper: ReturnType<typeof mountTab>, id: string) =>
+  wrapper.find(`[data-automation-id="${id}"]`).text()
+
+async function openInvoiceModal(wrapper: ReturnType<typeof mountTab>) {
+  await wrapper.find('[data-automation-id="JobFinishTab-create-invoice"]').trigger('click')
+  await flushPromises()
+}
+
+describe('JobFinishTab customer balance', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+    finishRetrieveMock.mockResolvedValue({ summary: summary(), checklist: checklist() })
+    invoicesRetrieveMock.mockResolvedValue({ invoices: [] })
+    costsSummaryRetrieveMock.mockResolvedValue(costSummary())
+  })
+
+  afterEach(resetFinishTab)
+
+  it('renders the subtotal, GST and Total to pay returned by the API', async () => {
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(text(wrapper, 'JobFinishTab-remaining-excl-gst')).toContain('600')
+    expect(text(wrapper, 'JobFinishTab-remaining-gst')).toContain('90')
+    expect(text(wrapper, 'JobFinishTab-total-to-pay')).toContain('1,150')
+  })
+
+  it('does not derive Total to pay from the other figures', async () => {
+    // A deliberately inconsistent payload: if the component recomputed the
+    // total it would show 1,290 rather than the server's answer.
+    finishRetrieveMock.mockResolvedValue({
+      summary: summary({ total_to_pay_incl_gst: 42 }),
+      checklist: checklist(),
+    })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(text(wrapper, 'JobFinishTab-total-to-pay')).toContain('42')
+  })
+
+  it('shows the outstanding invoice balance when Xero is still owed money', async () => {
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(text(wrapper, 'JobFinishTab-outstanding-incl-gst')).toContain('460')
+  })
+
+  it('hides the outstanding line when every invoice is paid', async () => {
+    finishRetrieveMock.mockResolvedValue({
+      summary: summary({ outstanding_invoiced_incl_gst: 0 }),
+      checklist: checklist(),
+    })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(wrapper.find('[data-automation-id="JobFinishTab-outstanding-incl-gst"]').exists()).toBe(
+      false,
+    )
+  })
+
+  it('offers no invoice action once the job is fully covered', async () => {
+    finishRetrieveMock.mockResolvedValue({
+      summary: summary({
+        valid_invoiced_excl_gst: 1000,
+        remaining_to_invoice_excl_gst: 0,
+        remaining_gst: 0,
+        remaining_to_invoice_incl_gst: 0,
+        total_to_pay_incl_gst: 0,
+      }),
+      checklist: checklist(),
+    })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(wrapper.find('[data-automation-id="JobFinishTab-create-invoice"]').exists()).toBe(false)
+    expect(wrapper.find('[data-automation-id="JobFinishTab-fully-invoiced"]').exists()).toBe(true)
+  })
+
+  it('shows an error instead of a zero balance when the load fails', async () => {
+    // Regression: the catch used to clear `loading` while leaving a fabricated
+    // zero summary on screen, so a failed request read as a settled job.
+    finishRetrieveMock.mockRejectedValue(new Error('boom'))
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(wrapper.find('[data-automation-id="JobFinishTab-load-error"]').exists()).toBe(true)
+    expect(wrapper.find('[data-automation-id="JobFinishTab-total-to-pay"]').exists()).toBe(false)
+    expect(wrapper.find('[data-automation-id="JobFinishTab-create-invoice"]').exists()).toBe(false)
+  })
+
+  it('reports an over-invoiced job instead of a negative balance', async () => {
+    finishRetrieveMock.mockResolvedValue({
+      summary: summary({
+        job_value_excl_gst: 800,
+        valid_invoiced_excl_gst: 1000,
+        remaining_to_invoice_excl_gst: 0,
+        remaining_gst: 0,
+        remaining_to_invoice_incl_gst: 0,
+        outstanding_invoiced_incl_gst: 0,
+        total_to_pay_incl_gst: 0,
+        over_invoiced_excl_gst: 200,
+      }),
+      checklist: checklist(),
+    })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect(text(wrapper, 'JobFinishTab-over-invoiced')).toContain('200')
+  })
+
+  it('re-reads the balance from the server after creating an invoice', async () => {
+    createInvoiceMock.mockResolvedValue({ success: true, messages: [] })
+    const wrapper = mountTab()
+    await flushPromises()
+    expect(finishRetrieveMock).toHaveBeenCalledTimes(1)
+
+    await openInvoiceModal(wrapper)
+    inModal('JobFinishTab-mode-invoice-full')!.click()
+    await flushPromises()
+
+    expect(createInvoiceMock).toHaveBeenCalled()
+    expect(finishRetrieveMock).toHaveBeenCalledTimes(2)
+  })
+
+  it('re-reads the balance from the server after deleting an invoice', async () => {
+    invoicesRetrieveMock.mockResolvedValue({
+      invoices: [
+        {
+          id: 'inv-1',
+          xero_id: 'xero-1',
+          number: 'INV-001',
+          status: 'AUTHORISED',
+          date: '2026-08-01',
+          total_excl_tax: 400,
+          online_url: 'https://xero.example/inv-1',
+        },
+      ],
+    })
+    deleteInvoiceMock.mockResolvedValue({})
+    const wrapper = mountTab()
+    await flushPromises()
+
+    await wrapper.get('[data-automation-id="JobInvoiceCard-delete-inv-1"]').trigger('click')
+    await flushPromises()
+
+    expect(deleteInvoiceMock).toHaveBeenCalled()
+    expect(finishRetrieveMock).toHaveBeenCalledTimes(2)
+  })
+})
+
+describe('JobFinishTab invoice modes', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+    finishRetrieveMock.mockResolvedValue({ summary: summary(), checklist: checklist() })
+    invoicesRetrieveMock.mockResolvedValue({ invoices: [] })
+    costsSummaryRetrieveMock.mockResolvedValue(costSummary())
+  })
+
+  afterEach(resetFinishTab)
+
+  it('keeps all three fixed-price modes available', async () => {
+    const wrapper = mountTab('fixed_price')
+    await flushPromises()
+    await openInvoiceModal(wrapper)
+
+    expect(inModal('JobFinishTab-mode-invoice-full')).not.toBeNull()
+    expect(inModal('JobFinishTab-mode-invoice-percent')).not.toBeNull()
+    expect(inModal('JobFinishTab-mode-invoice-amount')).not.toBeNull()
+  })
+
+  it('keeps both T&M modes available', async () => {
+    const wrapper = mountTab('time_materials')
+    await flushPromises()
+    await openInvoiceModal(wrapper)
+
+    expect(inModal('JobFinishTab-mode-invoice-costs-to-date')).not.toBeNull()
+    expect(inModal('JobFinishTab-mode-invoice-amount')).not.toBeNull()
+    expect(inModal('JobFinishTab-mode-invoice-percent')).toBeNull()
+  })
+
+  it('offers invoicing before the work is complete', async () => {
+    const wrapper = mountTab('fixed_price', 'approved')
+    await flushPromises()
+
+    expect(wrapper.find('[data-automation-id="JobFinishTab-create-invoice"]').exists()).toBe(true)
+    await openInvoiceModal(wrapper)
+    expect(modalText('JobFinishTab-mode-invoice-full')).toContain('in advance')
+  })
+
+  it('describes the remaining balance once the work is complete', async () => {
+    const wrapper = mountTab('fixed_price', 'recently_completed')
+    await flushPromises()
+    await openInvoiceModal(wrapper)
+
+    expect(modalText('JobFinishTab-mode-invoice-full')).toContain('remaining quote balance')
+  })
+})
diff --git a/frontend/src/components/job/__tests__/JobFinishTab.checklist.test.ts b/frontend/src/components/job/__tests__/JobFinishTab.checklist.test.ts
new file mode 100644
index 000000000..c91bb1450
--- /dev/null
+++ b/frontend/src/components/job/__tests__/JobFinishTab.checklist.test.ts
@@ -0,0 +1,158 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { flushPromises } from '@vue/test-utils'
+
+const {
+  finishRetrieveMock,
+  invoicesRetrieveMock,
+  costsSummaryRetrieveMock,
+  checklistUpdateMock,
+  toastErrorMock,
+} = vi.hoisted(() => ({
+  finishRetrieveMock: vi.fn(),
+  invoicesRetrieveMock: vi.fn(),
+  costsSummaryRetrieveMock: vi.fn(),
+  checklistUpdateMock: vi.fn(),
+  toastErrorMock: vi.fn(),
+}))
+
+vi.mock('@/api/client', () => ({
+  api: {
+    job_jobs_finish_retrieve: finishRetrieveMock,
+    job_jobs_invoices_retrieve: invoicesRetrieveMock,
+    job_jobs_costs_summary_retrieve: costsSummaryRetrieveMock,
+    job_jobs_finish_partial_update: checklistUpdateMock,
+    xero_create_invoice_create: vi.fn(),
+    xero_delete_invoice_destroy: vi.fn(),
+  },
+}))
+
+vi.mock('@/composables/useXeroConnection', () => ({
+  useXeroConnection: () => ({ xeroConnected: { value: true } }),
+}))
+
+vi.mock('vue-sonner', () => ({
+  toast: { error: toastErrorMock, success: vi.fn(), warning: vi.fn() },
+}))
+
+import JobFinishTab from '../JobFinishTab.vue'
+import {
+  CHECKLIST_ITEMS as ITEMS,
+  checklistState as checklist,
+  costSummary,
+  finishSummary as summary,
+  mountFinishTab,
+  resetFinishTab,
+} from './finishTabFixtures'
+
+const mountTab = (pricingMethodology = 'fixed_price') =>
+  mountFinishTab(JobFinishTab, pricingMethodology)
+
+const item = (wrapper: ReturnType<typeof mountTab>, key: string) =>
+  wrapper.find(`[data-automation-id="JobFinishTab-checklist-${key}"]`)
+
+describe('JobFinishTab completion checklist', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+    finishRetrieveMock.mockResolvedValue({ summary: summary(), checklist: checklist() })
+    invoicesRetrieveMock.mockResolvedValue({ invoices: [] })
+    costsSummaryRetrieveMock.mockResolvedValue(costSummary())
+  })
+
+  afterEach(resetFinishTab)
+
+  it('asks the same five questions whatever the pricing methodology', async () => {
+    for (const methodology of ['fixed_price', 'time_materials']) {
+      const wrapper = mountTab(methodology)
+      await flushPromises()
+
+      for (const key of ITEMS) {
+        expect(item(wrapper, key).exists(), `${methodology}/${key}`).toBe(true)
+      }
+
+      resetFinishTab()
+    }
+  })
+
+  it('warns that time and materials are what a T&M customer pays', async () => {
+    const wrapper = mountTab('time_materials')
+    await flushPromises()
+
+    expect(wrapper.find('[data-automation-id="JobFinishTab-tm-urgency"]').text()).toContain(
+      'before invoicing',
+    )
+  })
+
+  it('does not show the T&M urgency note on a quoted job', async () => {
+    const wrapper = mountTab('fixed_price')
+    await flushPromises()
+
+    expect(wrapper.find('[data-automation-id="JobFinishTab-tm-urgency"]').exists()).toBe(false)
+  })
+
+  it('reflects ticks already recorded on the job', async () => {
+    finishRetrieveMock.mockResolvedValue({
+      summary: summary(),
+      checklist: checklist({ foreman_signed_off: true }),
+    })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect((item(wrapper, 'foreman_signed_off').element as HTMLInputElement).checked).toBe(true)
+    expect((item(wrapper, 'released').element as HTMLInputElement).checked).toBe(false)
+  })
+
+  it('sends only the item that changed', async () => {
+    checklistUpdateMock.mockResolvedValue({
+      summary: summary(),
+      checklist: checklist({ materials_checked: true }),
+    })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    await item(wrapper, 'materials_checked').setValue(true)
+    await flushPromises()
+
+    expect(checklistUpdateMock).toHaveBeenCalledWith(
+      { materials_checked: true },
+      { params: { job_id: 'job-1' } },
+    )
+  })
+
+  it('sends false when a tick is withdrawn', async () => {
+    finishRetrieveMock.mockResolvedValue({
+      summary: summary(),
+      checklist: checklist({ released: true }),
+    })
+    checklistUpdateMock.mockResolvedValue({ summary: summary(), checklist: checklist() })
+    const wrapper = mountTab()
+    await flushPromises()
+
+    await item(wrapper, 'released').setValue(false)
+    await flushPromises()
+
+    expect(checklistUpdateMock).toHaveBeenCalledWith(
+      { released: false },
+      { params: { job_id: 'job-1' } },
+    )
+  })
+
+  it('reverts the box and warns the user when the save fails', async () => {
+    checklistUpdateMock.mockRejectedValue(new Error('nope'))
+    const wrapper = mountTab()
+    await flushPromises()
+
+    await item(wrapper, 'foreman_signed_off').setValue(true)
+    await flushPromises()
+
+    expect(toastErrorMock).toHaveBeenCalledWith('Failed to save checklist')
+    expect((item(wrapper, 'foreman_signed_off').element as HTMLInputElement).checked).toBe(false)
+  })
+
+  it('does not hide the invoice action behind the checklist', async () => {
+    const wrapper = mountTab()
+    await flushPromises()
+
+    expect((item(wrapper, 'foreman_signed_off').element as HTMLInputElement).checked).toBe(false)
+    expect(wrapper.find('[data-automation-id="JobFinishTab-create-invoice"]').exists()).toBe(true)
+  })
+})
diff --git a/frontend/src/components/job/__tests__/JobFinishTab.labourHours.test.ts b/frontend/src/components/job/__tests__/JobFinishTab.labourHours.test.ts
new file mode 100644
index 000000000..fb27400f7
--- /dev/null
+++ b/frontend/src/components/job/__tests__/JobFinishTab.labourHours.test.ts
@@ -0,0 +1,106 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { flushPromises } from '@vue/test-utils'
+
+const { finishRetrieveMock, invoicesRetrieveMock, costsSummaryRetrieveMock } = vi.hoisted(() => ({
+  finishRetrieveMock: vi.fn(),
+  invoicesRetrieveMock: vi.fn(),
+  costsSummaryRetrieveMock: vi.fn(),
+}))
+
+vi.mock('@/api/client', () => ({
+  api: {
+    job_jobs_finish_retrieve: finishRetrieveMock,
+    job_jobs_invoices_retrieve: invoicesRetrieveMock,
+    job_jobs_costs_summary_retrieve: costsSummaryRetrieveMock,
+    job_jobs_finish_partial_update: vi.fn(),
+    xero_create_invoice_create: vi.fn(),
+    xero_delete_invoice_destroy: vi.fn(),
+  },
+}))
+
+vi.mock('@/composables/useXeroConnection', () => ({
+  useXeroConnection: () => ({ xeroConnected: { value: true } }),
+}))
+
+vi.mock('vue-sonner', () => ({
+  toast: { error: vi.fn(), success: vi.fn(), warning: vi.fn() },
+}))
+
+import JobFinishTab from '../JobFinishTab.vue'
+import {
+  checklistState,
+  costSummary,
+  finishSummary,
+  mountFinishTab,
+  resetFinishTab,
+} from './finishTabFixtures'
+
+/**
+ * KAN-222: office staff read budget, used and remaining without doing the
+ * subtraction, and an overrun shows as a positive number rather than a negative
+ * remainder.
+ */
+async function mountWithHours(
+  hours: { estimate: number; quote: number; actual: number },
+  pricingMethodology = 'fixed_price',
+) {
+  costsSummaryRetrieveMock.mockResolvedValue(costSummary(hours))
+  const wrapper = mountFinishTab(JobFinishTab, pricingMethodology)
+  await flushPromises()
+  return wrapper
+}
+
+const tile = (wrapper: ReturnType<typeof mountFinishTab>, which: string) =>
+  wrapper.find(`[data-automation-id="JobFinishTab-labour-${which}"]`).text()
+
+describe('JobFinishTab labour hours', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+    finishRetrieveMock.mockResolvedValue({
+      summary: finishSummary(),
+      checklist: checklistState(),
+    })
+    invoicesRetrieveMock.mockResolvedValue({ invoices: [] })
+  })
+
+  afterEach(resetFinishTab)
+
+  it('reports budget, used and remaining-or-overrun', async () => {
+    const cases = [
+      { label: 'under budget', actual: 8, remaining: '4', heading: 'Hours remaining' },
+      { label: 'exactly on budget', actual: 12, remaining: '0', heading: 'Hours remaining' },
+      { label: 'over budget', actual: 15, remaining: '3', heading: 'Overrun' },
+    ]
+
+    for (const { label, actual, remaining, heading } of cases) {
+      const wrapper = await mountWithHours({ estimate: 10, quote: 12, actual })
+
+      expect(tile(wrapper, 'budget'), label).toBe('12')
+      expect(tile(wrapper, 'used'), label).toBe(String(actual))
+      expect(tile(wrapper, 'remaining'), label).toBe(remaining)
+      expect(tile(wrapper, 'remaining'), label).not.toContain('-')
+      expect(
+        wrapper.find('[data-automation-id="JobFinishTab-labour-hours"]').text(),
+        label,
+      ).toContain(heading)
+
+      resetFinishTab()
+    }
+  })
+
+  it('budgets from quote hours, falling back to estimate hours', async () => {
+    const cases = [
+      { label: 'fixed price with a quote', quote: 12, methodology: 'fixed_price', budget: '12' },
+      { label: 'fixed price, no quote', quote: 0, methodology: 'fixed_price', budget: '10' },
+      { label: 'time and materials', quote: 12, methodology: 'time_materials', budget: '10' },
+    ]
+
+    for (const { label, quote, methodology, budget } of cases) {
+      const wrapper = await mountWithHours({ estimate: 10, quote, actual: 5 }, methodology)
+
+      expect(tile(wrapper, 'budget'), label).toBe(budget)
+
+      resetFinishTab()
+    }
+  })
+})
diff --git a/frontend/src/components/job/__tests__/finishTabFixtures.ts b/frontend/src/components/job/__tests__/finishTabFixtures.ts
new file mode 100644
index 000000000..2162fea4c
--- /dev/null
+++ b/frontend/src/components/job/__tests__/finishTabFixtures.ts
@@ -0,0 +1,99 @@
+import { vi } from 'vitest'
+import { mount } from '@vue/test-utils'
+import type { Component } from 'vue'
+
+/** Shared fixtures for the JobFinishTab specs, which all mount the same tree. */
+
+const baseFinishSummary = {
+  job_value_excl_gst: 1000,
+  valid_invoiced_excl_gst: 400,
+  outstanding_invoiced_incl_gst: 460,
+  remaining_to_invoice_excl_gst: 600,
+  remaining_gst: 90,
+  remaining_to_invoice_incl_gst: 690,
+  total_to_pay_incl_gst: 1150,
+  over_invoiced_excl_gst: 0,
+}
+
+const baseChecklistState = {
+  foreman_signed_off: false,
+  timesheets_collected: false,
+  materials_checked: false,
+  customer_called: false,
+  released: false,
+}
+
+// Typed against the base shapes so a typo in an override is a compile error
+// rather than a silently ignored key that leaves the default in place.
+export const finishSummary = (overrides: Partial<typeof baseFinishSummary> = {}) => ({
+  ...baseFinishSummary,
+  ...overrides,
+})
+
+export const checklistState = (overrides: Partial<typeof baseChecklistState> = {}) => ({
+  ...baseChecklistState,
+  ...overrides,
+})
+
+export const CHECKLIST_ITEMS = [
+  'foreman_signed_off',
+  'timesheets_collected',
+  'materials_checked',
+  'customer_called',
+  'released',
+] as const
+
+export const costSummary = (
+  hours: { estimate: number; quote: number; actual: number } = {
+    estimate: 10,
+    quote: 12,
+    actual: 11,
+  },
+) => ({
+  estimate: { cost: 500, rev: 800, hours: hours.estimate, profitMargin: 60 },
+  quote: hours.quote > 0 ? { cost: 600, rev: 1000, hours: hours.quote, profitMargin: 66 } : null,
+  actual: { cost: 550, rev: 900, hours: hours.actual, profitMargin: 63 },
+})
+
+let mounted: ReturnType<typeof mount> | null = null
+
+export function mountFinishTab(
+  component: Component,
+  pricingMethodology = 'fixed_price',
+  jobStatus = 'in_progress',
+) {
+  // Mounted to document.body because the invoice modal renders through a
+  // reka-ui portal; resetFinishTab() must run in afterEach or the next test
+  // finds the previous modal.
+  mounted = mount(component, {
+    props: { jobId: 'job-1', pricingMethodology, jobStatus },
+    attachTo: document.body,
+  })
+  return mounted
+}
+
+export function resetFinishTab() {
+  mounted?.unmount()
+  mounted = null
+  document.body.innerHTML = ''
+}
+
+export const inModal = (id: string) =>
+  document.body.querySelector<HTMLElement>(`[data-automation-id="${id}"]`)
+
+export const modalText = (id: string) => {
+  const el = inModal(id)
+  if (!el) throw new Error(`${id} is not in the modal`)
+  return el.textContent ?? ''
+}
+
+/** The api-client mock shape every JobFinishTab spec needs. */
+export const finishTabApiMocks = () => ({
+  finishRetrieveMock: vi.fn(),
+  invoicesRetrieveMock: vi.fn(),
+  costsSummaryRetrieveMock: vi.fn(),
+  checklistUpdateMock: vi.fn(),
+  createInvoiceMock: vi.fn(),
+  deleteInvoiceMock: vi.fn(),
+  toastErrorMock: vi.fn(),
+})
diff --git a/frontend/src/composables/useJobFinancials.ts b/frontend/src/composables/useJobFinancials.ts
deleted file mode 100644
index 5e9d22370..000000000
--- a/frontend/src/composables/useJobFinancials.ts
+++ /dev/null
@@ -1,68 +0,0 @@
-import { ref, type Ref } from 'vue'
-import { api } from '../api/client'
-
-interface JobFinancials {
-  quoteTotal: number
-  actualTotal: number
-  invoiceTotal: number
-  toBeInvoiced: number
-}
-
-export function useJobFinancials(jobId: Ref<string>) {
-  const loading = ref(false)
-  const error = ref<string | null>(null)
-
-  const fetchJobFinancials = async (pricingMethodology?: string): Promise<JobFinancials> => {
-    loading.value = true
-    error.value = null
-
-    try {
-      // Fetch both cost summary and invoices in parallel
-      const [costSummary, invoicesResponse] = await Promise.all([
-        api.job_jobs_costs_summary_retrieve({
-          params: { job_id: jobId.value },
-        }),
-        api.job_jobs_invoices_retrieve({
-          params: { job_id: jobId.value },
-        }),
-      ])
-
-      // Extract amounts from cost summary
-      const quoteTotal = Number(costSummary.quote?.rev || 0)
-      const actualTotal = Number(costSummary.actual?.rev || 0)
-
-      // Calculate invoice total from invoices
-      const invoices = invoicesResponse.invoices || []
-      const invoiceTotal = invoices.reduce(
-        (sum: number, invoice: { total_excl_tax?: number }) =>
-          sum + Number(invoice.total_excl_tax || 0),
-        0,
-      )
-
-      // Calculate amount to invoice based on pricing methodology
-      const amountToInvoice =
-        pricingMethodology === 'fixed_price' && quoteTotal > 0 ? quoteTotal : actualTotal
-
-      const toBeInvoiced = Math.max(0, amountToInvoice - invoiceTotal)
-
-      return {
-        quoteTotal,
-        actualTotal,
-        invoiceTotal,
-        toBeInvoiced,
-      }
-    } catch (err) {
-      error.value = 'Failed to fetch job financial data'
-      console.error('Error fetching job financials:', err)
-      throw err
-    } finally {
-      loading.value = false
-    }
-  }
-
-  return {
-    fetchJobFinancials,
-    loading: loading as Readonly<Ref<boolean>>,
-    error: error as Readonly<Ref<string | null>>,
-  }
-}
diff --git a/frontend/src/constants/job-tabs.ts b/frontend/src/constants/job-tabs.ts
index ce19afadf..b6403253f 100644
--- a/frontend/src/constants/job-tabs.ts
+++ b/frontend/src/constants/job-tabs.ts
@@ -9,7 +9,7 @@ export const JobTabsSchema = z.enum([
   'quote',
   'actual',
   'financial',
-  'costAnalysis',
+  'finishJob',
   'jobSettings',
   'history',
   'attachments',
diff --git a/frontend/src/pages/jobs/[id]/(index).vue b/frontend/src/pages/jobs/[id]/(index).vue
index 97f0806ab..1969d2690 100644
--- a/frontend/src/pages/jobs/[id]/(index).vue
+++ b/frontend/src/pages/jobs/[id]/(index).vue
@@ -313,7 +313,6 @@ import { useRoute, useRouter } from 'vue-router'
 import { useJobsStore } from '@/stores/jobs'
 import { useJobTabs } from '@/composables/useJobTabs'
 import { useJobHeaderAutosave } from '@/composables/useJobHeaderAutosave'
-import { useJobFinancials } from '@/composables/useJobFinancials'
 import { useCompanyDefaultsStore } from '@/stores/companyDefaults'
 import { api } from '@/api/client'
 import { ArrowLeft, Printer } from 'lucide-vue-next'
@@ -450,7 +449,6 @@ const handlePricingMethodologyUpdate = (newMethod: string) => {
 }
 
 // Initialize the job financials composable
-const { fetchJobFinancials } = useJobFinancials(jobId)
 
 const handleRejectedChange = async () => {
   if (!jobHeader.value) return
@@ -484,14 +482,16 @@ const handleRejectedChange = async () => {
   } else {
     // CHECKING - validate no money to be invoiced
     try {
-      // Use the composable to fetch financial data
-      const financials = await fetchJobFinancials(jobHeader.value.pricing_methodology || undefined)
+      // The server owns this figure; it excludes voided invoices and respects
+      // the price cap, neither of which the old local calculation did.
+      const { summary } = await api.job_jobs_finish_retrieve({
+        params: { job_id: jobId.value },
+      })
 
-      // Check if there's money to be invoiced
-      if (financials.toBeInvoiced > 0) {
+      if (summary.remaining_to_invoice_excl_gst > 0) {
         // Show warning but allow proceeding
         const confirmed = confirm(
-          `Warning: Job has ${formatCurrency(financials.toBeInvoiced)} still to be invoiced. Are you sure you want to reject?`,
+          `Warning: Job has ${formatCurrency(summary.remaining_to_invoice_excl_gst)} still to be invoiced. Are you sure you want to reject?`,
         )
 
         if (!confirmed) {
diff --git a/frontend/tests/job/job-xero-invoice.spec.ts b/frontend/tests/job/job-xero-invoice.spec.ts
index 63e9da3d9..5cc818a9d 100644
--- a/frontend/tests/job/job-xero-invoice.spec.ts
+++ b/frontend/tests/job/job-xero-invoice.spec.ts
@@ -12,7 +12,7 @@ const getJobIdFromUrl = (url: string): string => {
 test.describe('job xero invoice', () => {
   test.setTimeout(120000)
 
-  test('create invoice in Xero from Actual tab', async ({
+  test('invoice a job from Finish Job and see the balance settle', async ({
     authenticatedPage: page,
     sharedEditJobUrl,
   }) => {
@@ -21,12 +21,18 @@ test.describe('job xero invoice', () => {
     await page.goto(sharedEditJobUrl)
     await page.waitForLoadState('networkidle')
 
-    await autoId(page, 'JobViewTabs-actual').click()
+    await autoId(page, 'JobViewTabs-finishJob').click()
 
-    const invoiceItems = page.locator('ul[role="list"] li')
+    // The balance is server-owned; the tab must render it before offering to invoice.
+    await expect(autoId(page, 'JobFinishTab-total-to-pay')).toBeVisible({ timeout: 10000 })
+    const remaining = autoId(page, 'JobFinishTab-remaining-excl-gst')
+    await expect(remaining).toBeVisible()
+    await expect(remaining).not.toHaveText(/\$0\.00/)
+
+    const invoiceItems = autoId(page, 'JobInvoiceCard-list').locator('li')
     const initialCount = await invoiceItems.count()
 
-    const createInvoiceButton = page.getByRole('button', { name: /Create .*Invoice/ })
+    const createInvoiceButton = autoId(page, 'JobFinishTab-create-invoice')
     await expect(createInvoiceButton).toBeVisible({ timeout: 10000 })
 
     const responsePromise = page.waitForResponse(
@@ -41,12 +47,9 @@ test.describe('job xero invoice', () => {
 
     await createInvoiceButton.click()
 
-    // Modal should appear — select the primary invoice mode
-    const invoiceRemainingButton = page.getByRole('button', {
-      name: /Invoice remaining quote/,
-    })
-    await expect(invoiceRemainingButton).toBeVisible({ timeout: 10000 })
-    await invoiceRemainingButton.click()
+    const invoiceFullButton = autoId(page, 'JobFinishTab-mode-invoice-full')
+    await expect(invoiceFullButton).toBeVisible({ timeout: 10000 })
+    await invoiceFullButton.click()
 
     const response = await responsePromise
     if (!response.ok()) {
@@ -57,5 +60,11 @@ test.describe('job xero invoice', () => {
     }
 
     await expect(invoiceItems).toHaveCount(initialCount + 1, { timeout: 20000 })
+
+    // KAN-323: a fully invoiced job shows nothing remaining and stops asking for
+    // another invoice, rather than offering a ceremonial $0 one.
+    await expect(remaining).toHaveText(/\$0\.00/, { timeout: 20000 })
+    await expect(autoId(page, 'JobFinishTab-fully-invoiced')).toBeVisible()
+    await expect(createInvoiceButton).toHaveCount(0)
   })
 })
diff --git a/mypy-baseline.txt b/mypy-baseline.txt
index 79b9805ed..51f22c9f9 100644
--- a/mypy-baseline.txt
+++ b/mypy-baseline.txt
@@ -380,35 +380,12 @@ apps/job/models/job.py:0: error: Function is missing a type annotation for one o
 apps/job/models/job.py:0: error: Incompatible types in assignment (expression has type "BooleanField[bool | Combinable, bool]", variable has type "bool")  [assignment]
 apps/job/models/job.py:0: error: Incompatible types in assignment (expression has type "BooleanField[bool | Combinable, bool]", variable has type "bool")  [assignment]
 apps/job/models/job.py:0: error: Incompatible types in assignment (expression has type "BooleanField[bool | Combinable, bool]", variable has type "bool")  [assignment]
-apps/job/models/job.py:0: error: Incompatible types in assignment (expression has type "BooleanField[bool | Combinable, bool]", variable has type "bool")  [assignment]
 apps/job/models/job.py:0: error: Incompatible types in assignment (expression has type "DateTimeField[str | datetime | date | Combinable | None, datetime | None]", variable has type "datetime | None")  [assignment]
 apps/job/models/job.py:0: error: Item "ForeignObjectRel" of "Field[Any, Any] | ForeignObjectRel" has no attribute "verbose_name"  [union-attr]
 apps/job/models/job.py:0: error: Item "None" of "str | None" has no attribute "rstrip"  [union-attr]
 apps/job/models/job.py:0: error: Missing type arguments for generic type "QuerySet"  [type-arg]
 apps/job/models/job.py:0: error: Returning Any from function declared to return "date | None"  [no-any-return]
 apps/job/models/job.py:0: error: Returning Any from function declared to return "date | None"  [no-any-return]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
-apps/job/models/job_event.py:0: error: "Callable[[dict[Any, Any]], str]" has no attribute "__func__"  [attr-defined]
 apps/job/models/job_event.py:0: error: Argument 1 to "_format_ordinal" has incompatible type "Any | None"; expected "int"  [arg-type]
 apps/job/models/job_event.py:0: error: Argument 1 to "_format_ordinal" has incompatible type "Any | None"; expected "int"  [arg-type]
 apps/job/models/job_event.py:0: error: Argument 1 to "_format_ordinal" has incompatible type "Any | None"; expected "int"  [arg-type]
@@ -439,7 +416,6 @@ apps/job/models/job_event.py:0: error: Returning Any from function declared to r
 apps/job/models/job_event.py:0: error: Returning Any from function declared to return "str"  [no-any-return]
 apps/job/models/job_event.py:0: error: Returning Any from function declared to return "str"  [no-any-return]
 apps/job/models/job_event.py:0: error: Returning Any from function declared to return "str"  [no-any-return]
-apps/job/models/job_event.py:0: error: Returning Any from function declared to return "str"  [no-any-return]
 apps/job/models/job_event.py:0: error: Unsupported left operand type for < ("None")  [operator]
 apps/job/models/job_event.py:0: note: Both left and right operands are unions
 apps/job/models/job_file.py:0: error: Argument 1 to "get_job_folder_path" has incompatible type "int"; expected "str"  [arg-type]
@@ -843,7 +819,6 @@ apps/job/tests/test_kanban_search.py:0: error: Function is missing a return type
 apps/job/tests/test_kanban_search.py:0: error: Incompatible type for "company" of "Invoice" (got "Company | None", expected "Company | Combinable")  [misc]
 apps/job/tests/test_kanban_search.py:0: error: Incompatible type for "company" of "Quote" (got "Company | None", expected "Company | Combinable")  [misc]
 apps/job/tests/test_mcp_tool_integration.py:0: error: Call to untyped function "get_queryset" in typed context  [no-untyped-call]
-apps/job/tests/test_mcp_tool_integration.py:0: error: Missing positional argument "query" in call to "search_products" of "QuotingTool"  [call-arg]
 apps/job/tests/test_modern_timesheet_views.py:0: error: Item "None" of "XeroPayItem | None" has no attribute "name"  [union-attr]
 apps/job/tests/test_pdf_goldens.py:0: error: Function is missing a return type annotation  [no-untyped-def]
 apps/job/tests/test_quote_modes.py:0: error: Call to untyped function "QuoteModeController" in typed context  [no-untyped-call]