@stapel/listings-react 0.3.0 → 0.4.0

package/src/headless/MyListings.tsx

@@ -11,28 +11,36 @@ import {
 } from "@stapel/core";
 import type { ActionAvailability, LoadState } from "@stapel/core";
 import type {
-  ListingCard,
-  ListingPageParams,
+  MyListingCard,
+  MyListingsParams,
   MyCounters,
 } from "../api/types.js";
-import { MY_LISTINGS_SOURCE_MISSING } from "../model/mineSource.js";
+import { defaultMyListingsSource } from "../model/mineSource.js";
 import type { MyListingsSource } from "../model/mineSource.js";
+import { useListingsApi } from "../model/context.js";
 import { useMyCounters } from "../model/queries.js";
 import { listingsQueryKeys, pageKey } from "../model/queryKeys.js";
-import { MY_LISTINGS_TABS } from "../model/status.js";
+import { MY_LISTINGS_TABS, MY_LISTINGS_UNTABBED_STATUSES } from "../model/status.js";
 import type { MyListingsTab } from "../model/status.js";
 import { LISTINGS_I18N_KEYS } from "../i18n/keys.js";
 import { useMandateGate } from "./useMandateGate.js";
 
 /**
- * The owner's dashboard — and the one screen in this pair whose rows the
- * backend cannot supply.
+ * The owner's dashboard.
  *
- * The counters are REAL and are shown. The rows come from an injected
- * `MyListingsSource`, and when there is none `rows` lands in the `failed` arm
- * carrying a NAMED reason rather than as an empty list. `model/mineSource.ts`
- * holds the whole argument and the upstream asks; this file is the bag over
- * it.
+ * Three counts and the rows behind them, both owner-scoped reads of
+ * stapel-listings: `my/counters` and — since 0.7.0 — `my/listings`. Until
+ * that release the rows had no route at all and this hook failed them with a
+ * named reason rather than rendering an empty grid; `model/mineSource.ts`
+ * keeps the argument and the seam that came out of it.
+ *
+ * TWO reads, not one, and the second one is the point of this file's shape:
+ * the three tabs are the SERVER's status groupings, and `blocked` — a
+ * moderation takedown — is in none of them, because `my/counters` counts it
+ * in none of them. A dashboard that only ever asked for a tab's statuses
+ * would hide exactly the listing whose owner most needs to know. So
+ * `blockedRows` is fetched beside them, off the same route, narrowed to
+ * whatever `MY_LISTINGS_UNTABBED_STATUSES` derives.
  */
 
 export interface MyListingsBag {
@@ -41,10 +49,16 @@ export interface MyListingsBag {
   setTab(tab: MyListingsTab): void;
   /** The three real counts. */
   readonly counters: LoadState<MyCounters>;
-  /** The rows for the current tab. `failed` with a named reason when no
-   * source is wired — never an empty list. */
-  readonly rows: LoadState<readonly ListingCard[]>;
-  readonly page: ListingPageParams;
+  /** The rows for the current tab. */
+  readonly rows: LoadState<readonly MyListingCard[]>;
+  /**
+   * The rows no tab folds in — a moderation takedown, today. Empty for
+   * almost every seller; when it is not, it is the most important thing on
+   * the screen. Never `failed` in a way that hides the tabs: this read is
+   * independent of `rows` and a skin renders it beside them.
+   */
+  readonly blockedRows: LoadState<readonly MyListingCard[]>;
+  readonly page: MyListingsParams;
   readonly nextPage: ActionAvailability;
   readonly prevPage: ActionAvailability;
   goNext(): void;
@@ -55,6 +69,8 @@ export interface MyListingsBag {
 }
 
 export interface UseMyListingsOptions {
+  /** Replace the contract's own `my/listings` read — a deployment that keeps
+   * its sellers' rows somewhere else. Absent: {@link defaultMyListingsSource}. */
   readonly source?: MyListingsSource;
   readonly initialTab?: MyListingsTab;
   readonly limit?: number;
@@ -68,27 +84,49 @@ export function useMyListings(
   const [tab, setTabState] = useState<MyListingsTab>(
     options.initialTab ?? "active"
   );
-  const [page, setPage] = useState<ListingPageParams>(
+  const [page, setPage] = useState<MyListingsParams>(
     options.limit !== undefined ? { limit: options.limit } : {}
   );
 
   const counters = useMyCounters();
-  const { source } = options;
+  const api = useListingsApi();
+  const injected = options.source;
+  const source = useMemo(
+    () => injected ?? defaultMyListingsSource(api),
+    [injected, api]
+  );
+  const ready = sessionReady && gate.available;
 
   const rowsQuery = useQuery({
     queryKey: listingsQueryKeys.mine(tab, pageKey(page)),
+    queryFn: ({ signal }) => source({ tab, page, signal }),
+    enabled: ready,
+    retry: false,
+  });
+
+  // The takedowns, off the same route and deliberately NOT paged: a seller
+  // with a page of blocked listings has a problem no "next" button improves,
+  // and this sits above a dashboard rather than being one.
+  const blockedQuery = useQuery({
+    queryKey: listingsQueryKeys.mineUntabbed(),
     queryFn: ({ signal }) =>
-      (source as MyListingsSource)({ tab, page, signal }),
-    enabled: source !== undefined && sessionReady && gate.available,
+      api.myListings({ status: MY_LISTINGS_UNTABBED_STATUSES }, { signal }),
+    enabled: ready && MY_LISTINGS_UNTABBED_STATUSES.length > 0,
     retry: false,
   });
 
-  const rows: LoadState<readonly ListingCard[]> = useMemo(() => {
-    if (source === undefined) return loadFailed(MY_LISTINGS_SOURCE_MISSING);
+  const rows: LoadState<readonly MyListingCard[]> = useMemo(() => {
     if (rowsQuery.status === "error") return loadFailed(rowsQuery.error);
     if (rowsQuery.data !== undefined) return loadReady(rowsQuery.data.items);
     return loadLoading();
-  }, [source, rowsQuery.status, rowsQuery.error, rowsQuery.data]);
+  }, [rowsQuery.status, rowsQuery.error, rowsQuery.data]);
+
+  const blockedRows: LoadState<readonly MyListingCard[]> = useMemo(() => {
+    if (MY_LISTINGS_UNTABBED_STATUSES.length === 0) return loadReady([]);
+    if (blockedQuery.status === "error") return loadFailed(blockedQuery.error);
+    if (blockedQuery.data !== undefined) return loadReady(blockedQuery.data.items);
+    return loadLoading();
+  }, [blockedQuery.status, blockedQuery.error, blockedQuery.data]);
 
   const envelope = rowsQuery.data;
 
@@ -110,6 +148,7 @@ export function useMyListings(
           ? loadReady(counters.data)
           : loadLoading(),
     rows,
+    blockedRows,
     page,
     nextPage:
       envelope?.has_next === true && envelope.next_anchor != null
@@ -133,6 +172,7 @@ export function useMyListings(
     refetch: () => {
       void counters.refetch();
       void rowsQuery.refetch();
+      void blockedQuery.refetch();
     },
   };
 }

package/src/i18n/es.ts

@@ -21,12 +21,14 @@ export { listingsErrorBundleEs } from "./generated/errors.es.gen.js";
 export const listingsI18nBundleEs: I18nDictionary = {
   ...listingsErrorBundleEs,
 
-  // ── the 9 stapel_listings-owned codes, pair-authored ─────────────────────
+  // ── the 10 stapel_listings-owned codes, pair-authored ────────────────────
   "error.400.category_required": "Hay que elegir una categoría",
   "error.400.image_required":
     "Para publicar el anuncio hace falta al menos una foto",
   "error.400.listing_feature_not_allowed":
     "El detalle «{feature}» no pertenece a esta categoría",
+  "error.400.listing_invalid_status_filter":
+    "Estado de anuncio desconocido: «{status}»",
   "error.400.publish_validation_failed":
     "El anuncio no pasó la revisión y no se publicó",
   "error.403.listing_not_owner": "Este anuncio no es tuyo",
@@ -91,7 +93,7 @@ export const listingsI18nBundleEs: I18nDictionary = {
   "listings.detail.photo_alt": "Foto {index} de {total}",
   "listings.detail.published_at": "Publicado el {date}",
   "listings.detail.expires_at": "A la venta hasta el {date}",
-  "listings.detail.stock": "Disponibles: {count}",
+  "listings.detail.stock": "Disponibles",
 
   "listings.compose.new_title": "Anuncio nuevo",
   "listings.compose.edit_title": "Editar el anuncio",
@@ -160,8 +162,13 @@ export const listingsI18nBundleEs: I18nDictionary = {
   "listings.mine.empty": "Aquí todavía no hay nada",
   "listings.mine.retry": "Reintentar",
   "listings.mine.counters_failed": "No pudimos contar tus anuncios",
-  "listings.mine.source_missing":
-    "Esta aplicación todavía no puede enumerar tus anuncios: el servicio de anuncios no tiene una lista limitada al propietario. Los contadores de abajo sí son reales.",
+  "listings.mine.empty.active": "No tienes nada publicado ni en revisión",
+  "listings.mine.empty.drafts": "Sin borradores: lo que empieces aparecerá aquí",
+  "listings.mine.empty.archived": "Nada archivado, pausado, caducado ni vendido todavía",
+  "listings.mine.blocked.title":
+    "Moderación retiró {count} de tus anuncios",
+  "listings.mine.blocked.load_failed":
+    "No pudimos comprobar si alguno de tus anuncios fue retirado",
   "listings.mine.live_under_review": "Publicado, cambios en revisión",
   "listings.mine.edit": "Editar",
   "listings.mine.archive": "Archivar",

package/src/i18n/generated/errors.es.gen.ts

@@ -8,7 +8,7 @@ import type { ListingsErrorCode } from "./errors.gen.js";
 /**
  * `es` texts for the backend error codes this catalog carries.
  *
- * PARTIAL, and deliberately typed to say so: 21 key(s) owned by
+ * PARTIAL, and deliberately typed to say so: 22 key(s) owned by
  * stapel_attributes, stapel_listings are absent, because that owner ships no locale catalog
  * (ERRORS_LOCALE_EXEMPT_OWNERS). English for them still comes from the registry
  * artifact via the en bundle; the pair layers its own authored `es`

package/src/i18n/generated/errors.gen.ts

@@ -66,6 +66,7 @@ export const LISTINGS_ERRORS = {
   "error.400.image_required": { status: 400, params: [], remediation: "fix_input", en: "At least one image is required to publish" },
   "error.400.invalid_ad_id": { status: 400, params: [], remediation: "fix_input", en: "Invalid advertisement ID" },
   "error.400.listing_feature_not_allowed": { status: 400, params: ["feature"], remediation: "fix_input", en: "Feature '{feature}' is not allowed for this category" },
+  "error.400.listing_invalid_status_filter": { status: 400, params: ["status"], remediation: "fix_input", en: "Unknown listing status '{status}'" },
   "error.400.publish_validation_failed": { status: 400, params: [], remediation: "fix_input", en: "Listing validation failed" },
   "error.400.validation_error": { status: 400, params: [], remediation: "fix_input", en: "Validation error" },
   "error.400.verification_failed": { status: 400, params: [], remediation: "verify", en: "Verification failed" },
@@ -135,6 +136,7 @@ export const LISTINGS_ERROR_CODES: readonly ListingsErrorCode[] = [
   "error.400.image_required",
   "error.400.invalid_ad_id",
   "error.400.listing_feature_not_allowed",
+  "error.400.listing_invalid_status_filter",
   "error.400.publish_validation_failed",
   "error.400.validation_error",
   "error.400.verification_failed",
@@ -206,6 +208,7 @@ export const listingsErrorBundleEn: Record<ListingsErrorCode, string> = {
   "error.400.image_required": "At least one image is required to publish",
   "error.400.invalid_ad_id": "Invalid advertisement ID",
   "error.400.listing_feature_not_allowed": "Feature '{feature}' is not allowed for this category",
+  "error.400.listing_invalid_status_filter": "Unknown listing status '{status}'",
   "error.400.publish_validation_failed": "Listing validation failed",
   "error.400.validation_error": "Validation error",
   "error.400.verification_failed": "Verification failed",

package/src/i18n/generated/errors.json

@@ -268,6 +268,15 @@
     "remediation": "fix_input",
     "en": "Feature '{feature}' is not allowed for this category"
   },
+  {
+    "code": "error.400.listing_invalid_status_filter",
+    "status": 400,
+    "params": [
+      "status"
+    ],
+    "remediation": "fix_input",
+    "en": "Unknown listing status '{status}'"
+  },
   {
     "code": "error.400.publish_validation_failed",
     "status": 400,

package/src/i18n/generated/errors.ru.gen.ts

@@ -8,7 +8,7 @@ import type { ListingsErrorCode } from "./errors.gen.js";
 /**
  * `ru` texts for the backend error codes this catalog carries.
  *
- * PARTIAL, and deliberately typed to say so: 21 key(s) owned by
+ * PARTIAL, and deliberately typed to say so: 22 key(s) owned by
  * stapel_attributes, stapel_listings are absent, because that owner ships no locale catalog
  * (ERRORS_LOCALE_EXEMPT_OWNERS). English for them still comes from the registry
  * artifact via the en bundle; the pair layers its own authored `ru`

package/src/i18n/keys.ts

@@ -84,6 +84,9 @@ export const LISTINGS_I18N_KEYS = {
   detailPhotoAlt: "listings.detail.photo_alt",
   detailPublishedAt: "listings.detail.published_at",
   detailExpiresAt: "listings.detail.expires_at",
+  /** The stock ROW'S LABEL. antd's `<Descriptions>` renders the colon and the
+   * quantity in the value cell, so this key carries no `{count}` — it used to,
+   * and the live page printed the placeholder. */
   detailStock: "listings.detail.stock",
 
   // ── composer ─────────────────────────────────────────────────────────────
@@ -141,8 +144,15 @@ export const LISTINGS_I18N_KEYS = {
   mineEmpty: "listings.mine.empty",
   mineRetry: "listings.mine.retry",
   mineCountersFailed: "listings.mine.counters_failed",
-  /** The upstream gap, named on screen — see `headless/MyListings.tsx`. */
-  mineSourceMissing: "listings.mine.source_missing",
+  /** Per-tab empties: "no drafts" and "nothing sold yet" are different
+   * sentences, and one generic "nothing here" says neither. */
+  mineEmptyActive: "listings.mine.empty.active",
+  mineEmptyDrafts: "listings.mine.empty.drafts",
+  mineEmptyArchived: "listings.mine.empty.archived",
+  /** The takedowns — the rows no tab folds in (`my/counters` counts them in
+   * none), shown outside the tabs so they cannot be missed. */
+  mineBlockedTitle: "listings.mine.blocked.title",
+  mineBlockedLoadFailed: "listings.mine.blocked.load_failed",
   mineLiveUnderReview: "listings.mine.live_under_review",
   mineEdit: "listings.mine.edit",
   mineArchive: "listings.mine.archive",
@@ -241,7 +251,7 @@ export const listingsI18nBundleEn: Record<string, string> = {
   "listings.detail.photo_alt": "Photo {index} of {total}",
   "listings.detail.published_at": "Published {date}",
   "listings.detail.expires_at": "Listed until {date}",
-  "listings.detail.stock": "{count} available",
+  "listings.detail.stock": "In stock",
 
   "listings.compose.new_title": "New listing",
   "listings.compose.edit_title": "Edit listing",
@@ -310,8 +320,13 @@ export const listingsI18nBundleEn: Record<string, string> = {
   "listings.mine.empty": "Nothing here yet",
   "listings.mine.retry": "Try again",
   "listings.mine.counters_failed": "We could not count your listings",
-  "listings.mine.source_missing":
-    "This app cannot list your own listings yet: the listings service has no owner-scoped list endpoint. Your counts below are real.",
+  "listings.mine.empty.active": "Nothing of yours is live or awaiting review",
+  "listings.mine.empty.drafts": "No drafts — anything you start appears here",
+  "listings.mine.empty.archived": "Nothing archived, paused, expired or sold yet",
+  "listings.mine.blocked.title":
+    "{count} of your listings were taken down by moderation",
+  "listings.mine.blocked.load_failed":
+    "We could not check whether any of your listings were taken down",
   "listings.mine.live_under_review":
     "Published, changes under review",
   "listings.mine.edit": "Edit",

package/src/i18n/ru.ts

@@ -37,12 +37,14 @@ export { listingsErrorBundleRu } from "./generated/errors.ru.gen.js";
 export const listingsI18nBundleRu: I18nDictionary = {
   ...listingsErrorBundleRu,
 
-  // ── the 9 stapel_listings-owned codes, pair-authored ─────────────────────
+  // ── the 10 stapel_listings-owned codes, pair-authored ────────────────────
   "error.400.category_required": "Нужно выбрать категорию",
   "error.400.image_required":
     "Чтобы опубликовать объявление, добавьте хотя бы одно фото",
   "error.400.listing_feature_not_allowed":
     "Характеристика «{feature}» не относится к этой категории",
+  "error.400.listing_invalid_status_filter":
+    "Неизвестный статус объявления: «{status}»",
   "error.400.publish_validation_failed":
     "Объявление не прошло проверку и не опубликовано",
   "error.403.listing_not_owner": "Это не ваше объявление",
@@ -107,7 +109,7 @@ export const listingsI18nBundleRu: I18nDictionary = {
   "listings.detail.photo_alt": "Фото {index} из {total}",
   "listings.detail.published_at": "Опубликовано {date}",
   "listings.detail.expires_at": "В продаже до {date}",
-  "listings.detail.stock": "В наличии: {count}",
+  "listings.detail.stock": "В наличии",
 
   "listings.compose.new_title": "Новое объявление",
   "listings.compose.edit_title": "Редактирование объявления",
@@ -177,8 +179,13 @@ export const listingsI18nBundleRu: I18nDictionary = {
   "listings.mine.empty": "Здесь пока пусто",
   "listings.mine.retry": "Повторить",
   "listings.mine.counters_failed": "Не удалось посчитать ваши объявления",
-  "listings.mine.source_missing":
-    "Приложение пока не может перечислить ваши объявления: у сервиса объявлений нет списка, ограниченного владельцем. Счётчики ниже — настоящие.",
+  "listings.mine.empty.active": "Ни одно ваше объявление не опубликовано и не ждёт проверки",
+  "listings.mine.empty.drafts": "Черновиков нет — всё начатое появится здесь",
+  "listings.mine.empty.archived": "Пока ничего не в архиве, не снято, не истекло и не продано",
+  "listings.mine.blocked.title":
+    "Модерация сняла ваших объявлений: {count}",
+  "listings.mine.blocked.load_failed":
+    "Не удалось проверить, снимала ли модерация ваши объявления",
   "listings.mine.live_under_review": "Опубликовано, правки на проверке",
   "listings.mine.edit": "Редактировать",
   "listings.mine.archive": "В архив",

package/src/index.ts

@@ -38,9 +38,10 @@
  *    a grey button and never a hidden one.
  * 4. **An absence is never rendered as a zero.** A soft-deleted listing says
  *    it was removed rather than 404-ing like a typo; a stored feature this
- *    build cannot read is counted rather than dropped; and the owner's own
- *    listings — which stapel-listings 0.6.1 has NO endpoint for — arrive as a
- *    named failure rather than an empty grid (`headless/MyListings.tsx`).
+ *    build cannot read is counted rather than dropped; an empty dashboard tab
+ *    says WHICH emptiness it is; and a moderation takedown — the one status
+ *    `my/counters` counts in no tab at all — is shown outside the tabs rather
+ *    than falling out of the screen (`headless/MyListings.tsx`).
  *
  * ── The seams, and why they are not imports ────────────────────────────────
  *
@@ -88,7 +89,10 @@ export type {
   ListingPageParams,
   ListingStatusInfo,
   MyCounters,
+  MyListingCard,
+  MyListingsParams,
   PaginatedListingCards,
+  PaginatedMyListingCards,
   PublishResponse,
   Schemas,
 } from "./api/types.js";
@@ -97,6 +101,7 @@ export type {
 export {
   MY_LISTINGS_TABS,
   MY_LISTINGS_TAB_STATUSES,
+  MY_LISTINGS_UNTABBED_STATUSES,
   isPubliclyVisible,
   lifecycleCaption,
   listingStatusView,
@@ -112,9 +117,15 @@ export type {
 } from "./model/status.js";
 export { LISTING_TRANSITIONS, canDelete, canTransition } from "./model/transitions.js";
 
-// ── model: the owner-list gap, named ─────────────────────────────────────────
-export { MY_LISTINGS_SOURCE_MISSING } from "./model/mineSource.js";
+// ── model: the owner's own rows ──────────────────────────────────────────────
+export { defaultMyListingsSource } from "./model/mineSource.js";
 export type { MyListingsSource } from "./model/mineSource.js";
+export {
+  myListingImages,
+  myListingPrice,
+  myListingTitle,
+  showsDraft,
+} from "./model/mine.js";
 
 // ── model: the draft twin ────────────────────────────────────────────────────
 export {

package/src/model/mine.ts

@@ -0,0 +1,70 @@
+/**
+ * What an owner's row SAYS, when half its fields may still be a draft.
+ *
+ * A listing carries every user-editable value twice: the `*_draft` twin the
+ * owner is writing, and the published field `publish` promotes it onto. The
+ * public card reads the published half — correct for a shop window, and empty
+ * for a listing that has never been published. A drafts tab built on it is a
+ * column of blank rows.
+ *
+ * So the rule, in one place rather than at three call sites: **show the
+ * published value when there is one, and the draft otherwise.** Not "always
+ * the draft" — a live listing being edited must go on reading as what
+ * strangers currently see, or the seller cannot tell what is on the shelf
+ * from what they are about to put there.
+ *
+ * Pure, so `test/mine.test.tsx` asserts the table directly.
+ */
+import type { MyListingCard } from "../api/types.js";
+
+/** Is this string field's published half absent? `""` counts: the model
+ * declares `title = CharField(blank=True, default="")`, so an unpublished
+ * listing carries the empty string, not `null`. */
+function empty(value: string | null | undefined): boolean {
+  return value === null || value === undefined || value === "";
+}
+
+/**
+ * The row's heading: the published title, the draft title, or `undefined`.
+ *
+ * `undefined` and not the id: a caller that wants "Listing #41" as a last
+ * resort can say so, and one that wants to render a placeholder can do that
+ * instead. Inventing the fallback here would take the choice away.
+ */
+export function myListingTitle(row: MyListingCard): string | undefined {
+  if (!empty(row.title)) return row.title as string;
+  if (!empty(row.title_draft)) return row.title_draft as string;
+  return undefined;
+}
+
+/**
+ * The row's price, published half first. A decimal STRING on the wire
+ * (`DecimalField` → `"200.00"`), passed through untouched — formatting it is
+ * a skin's job and rounding it here would lose the trailing zero a currency
+ * needs.
+ */
+export function myListingPrice(row: MyListingCard): string | undefined {
+  if (!empty(row.price)) return row.price as string;
+  if (!empty(row.price_draft)) return row.price_draft as string;
+  return undefined;
+}
+
+/** The row's images, published half first — `[]` when neither has any. */
+export function myListingImages(row: MyListingCard): readonly string[] {
+  const published = row.images;
+  if (published !== null && published !== undefined && published.length > 0) {
+    return published;
+  }
+  return row.images_draft ?? [];
+}
+
+/**
+ * Is what this row shows the DRAFT rather than what strangers see?
+ *
+ * The half of the rule a person is owed on screen: a row whose title came off
+ * the draft twin is showing something nobody else can read yet, and a skin
+ * that did not mark it would be quietly claiming otherwise.
+ */
+export function showsDraft(row: MyListingCard): boolean {
+  return empty(row.title) && !empty(row.title_draft);
+}

package/src/model/mineSource.ts

@@ -1,55 +1,60 @@
 /**
- * THE GAP: stapel-listings cannot list a person their own listings.
+ * THE GAP, AND THE ROUTE THAT CLOSED IT.
  *
- * This is the one place in the pair where a contract is missing rather than
- * awkward, so it gets its own module and its own name.
+ * This file used to open "stapel-listings cannot list a person their own
+ * listings", and that was the only place in the pair where a contract was
+ * missing rather than awkward. `GET /listings/` answers `qs.published()` and
+ * takes no owner parameter (`views.ListingViewSet.get_queryset`): it returns
+ * the whole marketplace's shop window and can be narrowed to nobody. The only
+ * owner-scoped reads were `my/counters` (three integers) and `my/favorites`,
+ * so a seller's own DRAFTS were unreachable by any call the contract offered
+ * — and drafts are not indexed either, so routing around it through
+ * `@stapel/search-react`'s `owner=` would have returned the published subset
+ * and quietly called it "everything". The dashboard shipped with a
+ * host-injected source and NAMED the absence on screen, because "we cannot
+ * ask" and "you have no listings" are different sentences.
  *
- * `GET /listings/` answers `qs.published()` and takes no owner parameter
- * (`views.ListingViewSet.get_queryset`): it returns the whole marketplace's
- * shop window and can be narrowed to nobody. The only owner-scoped reads in
- * the module are `my/counters` (three integers) and `my/favorites`. So a
- * seller's own DRAFTS are unreachable by any call this contract offers — and
- * drafts are not indexed either, so routing around it through
- * `@stapel/search-react`'s `owner=` would return the published subset and
- * quietly call it "everything".
+ * **stapel-listings 0.7.0 answers it**: `GET /listings/my/listings/`, the
+ * caller's own rows in every status, `?status=` for a tab's set, the same
+ * `IDAnchorPagination` envelope the other two owner reads use. So
+ * {@link defaultMyListingsSource} is what the dashboard runs on now, and
+ * there is no failure state left to name — the missing-source error and its
+ * i18n key are gone rather than kept as a comment about a thing that no
+ * longer happens.
  *
- * The storefront spec assumed `GET /listings/` was "for mine" (§4.1). It is
- * not, and this pair records the gap rather than papering over it.
- *
- * Two upstream asks, in preference order:
- *  1. an `?owner=me` filter (or a `my/listings` action) on the list endpoint,
- *     with the same `IDAnchorPagination` envelope the other two use;
- *  2. failing that, a `status` filter on it, so at least the published subset
- *     can be narrowed without a search module.
- *
- * Until then a host injects a {@link MyListingsSource} and the dashboard
- * works completely; without one it shows the real counters and NAMES the
- * absence, because "we cannot ask" and "you have no listings" are different
- * sentences.
+ * {@link MyListingsSource} stays, as a seam and not as a workaround: a
+ * deployment that keeps its sellers' rows somewhere else (a read model, a
+ * legacy table, a host that has already fetched the page) hands one in and
+ * the dashboard renders it unchanged. What it must return is one page of the
+ * owner card — including the `moderation_status` and the `*_draft` twins,
+ * which is why the type is {@link PaginatedMyListingCards} and not the public
+ * card envelope.
  */
-import { StapelApiError } from "@stapel/core";
-import type { ListingPageParams, PaginatedListingCards } from "../api/types.js";
+import type { ListingsApi } from "../api/listingsApi.js";
+import type { MyListingsParams, PaginatedMyListingCards } from "../api/types.js";
 import type { MyListingsTab } from "./status.js";
-import { LISTINGS_I18N_KEYS } from "../i18n/keys.js";
+import { MY_LISTINGS_TAB_STATUSES } from "./status.js";
 
-/** One page of the caller's own listings, however the host can get them. */
+/** One page of the caller's own listings, however the host gets them. */
 export type MyListingsSource = (args: {
   readonly tab: MyListingsTab;
-  readonly page: ListingPageParams;
+  readonly page: MyListingsParams;
   readonly signal?: AbortSignal;
-}) => Promise<PaginatedListingCards>;
+}) => Promise<PaginatedMyListingCards>;
 
 /**
- * The error a dashboard without a source reports.
+ * The source the dashboard uses unless a host replaces it: the contract's own
+ * route, narrowed to the tab's statuses.
  *
- * `status: 0` on purpose: this is a fact about the deployment's wiring, not
- * something a server said, and a client-side refusal must never be
- * indistinguishable from one that came over the wire.
+ * The tab → statuses table is `MY_LISTINGS_TAB_STATUSES`, which is a copy of
+ * the SERVER's grouping in `views.my_counters`. That is the whole point: the
+ * rows this fetches and the count on the tab beside them are the same set,
+ * asserted upstream (`tests/test_my_listings.py`) and here.
  */
-export const MY_LISTINGS_SOURCE_MISSING: StapelApiError = new StapelApiError({
-  code: LISTINGS_I18N_KEYS.mineSourceMissing,
-  message:
-    "No MyListingsSource was supplied, and stapel-listings has no " +
-    "owner-scoped list endpoint to fall back on.",
-  status: 0,
-});
+export function defaultMyListingsSource(api: ListingsApi): MyListingsSource {
+  return ({ tab, page, signal }) =>
+    api.myListings(
+      { ...page, status: MY_LISTINGS_TAB_STATUSES[tab] },
+      signal !== undefined ? { signal } : {}
+    );
+}

package/src/model/mutations.ts

@@ -27,10 +27,13 @@ import { listingsQueryKeys } from "./queryKeys.js";
  * ── The invalidation rule ──────────────────────────────────────────────────
  *
  * A write that can move `status` or `moderation_status` invalidates the
- * DETAIL, the STATUS probe and the COUNTERS — the counters because their
- * three tabs are defined by status sets (`views.my_counters`), so a publish
- * silently moves a row from "drafts" to "active" and a dashboard that did not
- * refetch would show the old number beside the new row.
+ * DETAIL, the STATUS probe, the COUNTERS and the owner's ROWS — the counters
+ * because their three tabs are defined by status sets (`views.my_counters`),
+ * so a publish silently moves a row from "drafts" to "active" and a dashboard
+ * that did not refetch would show the old number beside the new row; the rows
+ * for the same reason, since stapel-listings 0.7.0 gave them a route and they
+ * are narrowed by exactly those status sets. Invalidating one and not the
+ * other is the shape of the bug where the badge says 2 and the tab shows 3.
  *
  * Nothing here is optimistic except the favourite toggle, and that exception
  * is argued at its own hook.
@@ -46,6 +49,7 @@ function invalidateListing(queryClient: QueryClient, id: number): void {
   void queryClient.invalidateQueries({
     queryKey: listingsQueryKeys.validateDraft(id),
   });
+  void queryClient.invalidateQueries({ queryKey: listingsQueryKeys.allMine() });
 }
 
 /** Start a draft. The server forces `owner` and `status=draft`. */
@@ -63,6 +67,9 @@ export function useCreateDraft(): UseMutationResult<
       void queryClient.invalidateQueries({
         queryKey: listingsQueryKeys.myCounters(),
       });
+      void queryClient.invalidateQueries({
+        queryKey: listingsQueryKeys.allMine(),
+      });
     },
   });
 }