printedparts stage 14: retire/restore in the UI, dashless item codes

Retire button with confirmation on the detail page (item leaves the
storefront and the kiosk rejects its code; ledger history and label
survive), Restore on retired items, and an Include-retired list toggle
with a badge. Restore is its own permission-gated POST - the generic
update still cannot flip isactive. New codes mint as WJRP0042 style
without the dash; existing codes are immutable bin labels and keep
their form.

docs/PLUGIN-LAB-PRINTEDPARTS.md

@@ -335,6 +335,18 @@ docs-drift guard again).
 4. Test: active user's email + free-text merge deduped, inactive user
    skipped (`test_alert_recipients_merge_users_and_freetext`).
 
+## Stage 14 (extension) - retire/restore in the UI, dashless codes
+
+Field feedback stage: the soft-delete endpoint existed with no button, and
+the site wanted `WJRP0042`, not `WJRP-0042`.
+1. Detail gains Retire (confirm dialog; item leaves the storefront and the
+   kiosk 404s its code, history and label intact) and Restore; the list
+   gains an Include-retired toggle (`?active=false`) with a Retired badge.
+   Restore is its own POST gated by printedparts.delete - PUT deliberately
+   cannot flip isactive.
+2. Minting drops the dash: `f'{prefix}{id:04d}'`. Existing items keep their
+   codes - itemcode is an immutable label once printed on a bin.
+
 ---
 
 ## Where each pattern lives (cheat sheet)

frontend/src/api/index.js

@@ -1144,6 +1144,9 @@ export const printedpartsApi = {
   remove(printeditemid) {
     return api.delete(`/printedparts/items/${printeditemid}`)
   },
+  restore(printeditemid) {
+    return api.post(`/printedparts/items/${printeditemid}/restore`)
+  },
   uploadImage(printeditemid, file) {
     const formData = new FormData()
     formData.append('file', file)

frontend/src/views/printedparts/PrintedItemDetail.vue

@@ -14,6 +14,7 @@
               {{ item.quantityonhand }} on hand
             </span>
             <span v-if="item.islowstock" class="badge badge-warning">Low stock</span>
+            <span v-if="!item.isactive" class="badge badge-secondary">Retired</span>
           </div>
           <div class="hero-details">
             <p v-if="item.itemdescription">{{ item.itemdescription }}</p>
@@ -29,6 +30,10 @@
                          class="btn btn-secondary btn-sm">Edit</router-link>
             <router-link :to="`/print/printedparts-labels?item=${item.printeditemid}`"
                          class="btn btn-secondary btn-sm">Bin Label</router-link>
+            <button v-if="item.isactive" class="btn btn-danger btn-sm"
+                    @click="retireItem">Retire</button>
+            <button v-else class="btn btn-primary btn-sm"
+                    @click="restoreItem">Restore</button>
           </div>
         </div>
       </div>
@@ -197,6 +202,28 @@ async function submitLedger() {
   }
 }
 
+async function retireItem() {
+  if (!window.confirm(
+      `Retire ${item.value.itemname}? It leaves the storefront and kiosk; `
+      + 'history and the bin label stay, and it can be restored later.')) return
+  try {
+    await printedpartsApi.remove(item.value.printeditemid)
+    const response = await printedpartsApi.get(item.value.printeditemid)
+    item.value = response.data.data
+  } catch (retireError) {
+    console.error('Retire failed:', retireError)
+  }
+}
+
+async function restoreItem() {
+  try {
+    const response = await printedpartsApi.restore(item.value.printeditemid)
+    item.value = response.data.data
+  } catch (restoreError) {
+    console.error('Restore failed:', restoreError)
+  }
+}
+
 function formatDate(value) {
   if (!value) return '-'
   return new Date(value).toLocaleString()

frontend/src/views/printedparts/PrintedItemsList.vue

@@ -22,6 +22,10 @@
         <input v-model="lowstockOnly" type="checkbox" @change="loadItems" />
         Low stock only
       </label>
+      <label class="lowstock-filter">
+        <input v-model="includeRetired" type="checkbox" @change="loadItems" />
+        Include retired
+      </label>
     </div>
 
     <div class="card">
@@ -56,7 +60,10 @@
                   />
                 </td>
                 <td>{{ item.itemcode || '-' }}</td>
-                <td>{{ item.itemname }}</td>
+                <td>
+                  {{ item.itemname }}
+                  <span v-if="!item.isactive" class="badge badge-secondary">Retired</span>
+                </td>
                 <td>
                   <span :class="['badge', item.islowstock ? 'badge-danger' : 'badge-success']">
                     {{ item.quantityonhand }}
@@ -92,6 +99,7 @@ import { withBase } from '../../utils/basePath'
 const items = ref([])
 const loading = ref(true)
 const lowstockOnly = ref(false)
+const includeRetired = ref(false)
 const { page, search, setPage, setSearch } = useListQuery({ onChange: loadItems })
 const totalPages = ref(1)
 const perPage = ref(20)
@@ -106,6 +114,7 @@ async function loadItems() {
     const params = { page: page.value, perpage: perPage.value }
     if (search.value) params.search = search.value
     if (lowstockOnly.value) params.lowstock = 'true'
+    if (includeRetired.value) params.active = 'false'
     const response = await printedpartsApi.list(params)
     items.value = response.data.data || []
     totalPages.value = response.data.meta?.pagination?.totalpages || 1

frontend/src/views/settings/PrintedPartsSettings.vue

@@ -13,7 +13,7 @@
         <input v-model="values.printedparts_code_prefix" type="text"
                class="form-control" maxlength="8" />
         <p class="field-hint">
-          New items mint codes like {{ values.printedparts_code_prefix || '3DP' }}-0042.
+          New items mint codes like {{ values.printedparts_code_prefix || '3DP' }}0042.
           Changing it does not rename existing items.
         </p>
       </div>

plugins/printedparts/api/routes.py

@@ -91,7 +91,7 @@ def _imagedir():
 def _mint_itemcode(item):
     """Set itemcode from the configured prefix + the flushed row id."""
     prefix = Setting.get('printedparts_code_prefix') or '3DP'
-    item.itemcode = f'{prefix}-{item.printeditemid:04d}'
+    item.itemcode = f'{prefix}{item.printeditemid:04d}'
 
 
 @printedparts_bp.route('/items', methods=['POST'])
@@ -159,6 +159,20 @@ def delete_item(item_id: int):
     return success_response(message='Printed item retired')
 
 
+@printedparts_bp.route('/items/<int:item_id>/restore', methods=['POST'])
+@jwt_required()
+@require_permission('printedparts.delete')
+def restore_item(item_id: int):
+    """Bring a retired item back; code, photo, and history are intact."""
+    item = db.session.get(PrintedItem, item_id)
+    if not item:
+        return error_response(ErrorCodes.NOT_FOUND,
+                              f'Printed item {item_id} not found', http_code=404)
+    item.isactive = True
+    db.session.commit()
+    return success_response(item.to_dict(), message='Printed item restored')
+
+
 # --- item image: the models.py upload/serve/delete trio ---------------------
 
 @printedparts_bp.route('/items/<int:item_id>/image', methods=['POST'])

tests/test_plugins/test_printedparts_ledger.py

@@ -39,7 +39,7 @@ def test_create_mints_itemcode(client, auth_headers):
                            headers=auth_headers)
     assert response.status_code == 201
     data = response.get_json()['data']
-    assert data['itemcode'] == f"3DP-{data['printeditemid']:04d}"
+    assert data['itemcode'] == f"3DP{data['printeditemid']:04d}"
     assert data['quantityonhand'] == 0
 
 
@@ -240,3 +240,23 @@ def test_alert_recipients_merge_users_and_freetext(client, auth_headers, app,
     assert take.status_code == 200          # 4 on hand: crossed threshold 5
 
     assert captured['to'] == ['lead@site.test', 'extra@site.test']
+
+
+def test_retire_hides_and_restore_returns(client, auth_headers, item):
+    """Retire drops the item from the default list and the kiosk; restore
+    brings it back with history intact."""
+    assert client.delete(f'/api/printedparts/items/{item}',
+                         headers=auth_headers).status_code == 200
+
+    listed = client.get('/api/printedparts/items').get_json()['data']
+    assert all(row['printeditemid'] != item for row in listed)
+    kiosk = client.get('/api/printedparts/kiosk/item/3DP-9001')
+    assert kiosk.status_code == 404
+
+    including = client.get('/api/printedparts/items?active=false')
+    assert any(row['printeditemid'] == item
+               for row in including.get_json()['data'])
+
+    assert client.post(f'/api/printedparts/items/{item}/restore',
+                       headers=auth_headers).status_code == 200
+    assert client.get('/api/printedparts/kiosk/item/3DP-9001').status_code == 200