From 7c3838ba1f6be2ca80d35a50be7b6e79b3c87f0a Mon Sep 17 00:00:00 2001
From: Rassl <rasul.sahibjanli@gmail.com>
Date: Fri, 8 May 2026 02:51:52 +0400
Subject: [PATCH] feat: update my content panel to match new contract

---
 src/components/layout/my-content-panel.tsx | 10 ++--
 src/lib/__tests__/my-content-page.test.tsx | 69 +++++++++-------------
 2 files changed, 33 insertions(+), 46 deletions(-)

diff --git a/src/components/layout/my-content-panel.tsx b/src/components/layout/my-content-panel.tsx
index 08feaf2..d518d03 100644
--- a/src/components/layout/my-content-panel.tsx
+++ b/src/components/layout/my-content-panel.tsx
@@ -37,12 +37,11 @@ interface ContentResponse {
 }
 
 export function MyContentPanel({ onClose }: { onClose: () => void }) {
-  const { pubKey, routeHint, isAdmin } = useUserStore()
+  const { pubKey, isAdmin } = useUserStore()
   const schemas = useSchemaStore((s) => s.schemas)
   const openModal = useModalStore((s) => s.open)
   const setHoveredNode = useGraphStore((s) => s.setHoveredNode)
   const setSidebarSelectedNode = useGraphStore((s) => s.setSidebarSelectedNode)
-  const userFullPubkey = pubKey && routeHint ? `${pubKey}_${routeHint}` : pubKey
   const mocksEnabled = isMocksEnabled()
   const [nodes, setNodes] = useState<GraphNode[]>([])
   const [totalProcessing, setTotalProcessing] = useState(0)
@@ -194,7 +193,8 @@ export function MyContentPanel({ onClose }: { onClose: () => void }) {
             ) : (
               <div className="py-1">
                 {nodes.map((node, i) => {
-                  const canDelete = isAdmin || node.properties?.pubkey === userFullPubkey
+                  // /v2/content is server-filtered to the caller's content, so every node
+                  // here is the user's — always deletable, never self-boostable.
                   const isConfirming = deletingId === node.ref_id
 
                   const handleConfirmDelete = async () => {
@@ -212,10 +212,10 @@ export function MyContentPanel({ onClose }: { onClose: () => void }) {
                           onClick={() => { setSelectedNode(node); setSidebarSelectedNode(node) }}
                           onMouseEnter={() => setHoveredNode(node)}
                           onMouseLeave={() => setHoveredNode(null)}
-                          hideBoost={isAdmin || node.properties?.pubkey === userFullPubkey}
+                          hideBoost={true}
                           isAdmin={isAdmin}
                         />
-                        {canDelete && !isConfirming && (
+                        {!isConfirming && (
                           <button
                             className="absolute right-2 top-1/2 -translate-y-1/2 opacity-0 group-hover:opacity-100 transition-opacity text-muted-foreground hover:text-destructive"
                             onClick={(e) => { e.stopPropagation(); setDeletingId(node.ref_id) }}
diff --git a/src/lib/__tests__/my-content-page.test.tsx b/src/lib/__tests__/my-content-page.test.tsx
index d384fa9..59b00ba 100644
--- a/src/lib/__tests__/my-content-page.test.tsx
+++ b/src/lib/__tests__/my-content-page.test.tsx
@@ -114,7 +114,7 @@ describe("MyContentPanel", () => {
     expect(screen.queryByText(/still processing/i)).not.toBeInTheDocument()
   })
 
-  it("renders read-only boost amount when node has boost property", async () => {
+  it("never renders boost UI in MyContent (every node here is the user's own content)", async () => {
     mockApiGet.mockResolvedValue({
       nodes: [
         {
@@ -128,9 +128,9 @@ describe("MyContentPanel", () => {
     })
     render(<MyContentPanel onClose={() => {}} />)
     await waitFor(() => {
-      expect(screen.getByText("150")).toBeInTheDocument()
-      expect(screen.getByText("sats")).toBeInTheDocument()
+      expect(screen.getByText("Bitcoin is freedom")).toBeInTheDocument()
     })
+    expect(screen.queryByText("sats")).not.toBeInTheDocument()
   })
 
   it("renders no boost display when boost is absent", async () => {
@@ -164,19 +164,23 @@ describe("MyContentPanel", () => {
     )
   })
 
-  it("hides boost sats display when node pubkey matches user pubKey (contributor)", async () => {
+  it("hides boost sats display when node has owner_reference_id (contributor)", async () => {
     mockApiGet.mockResolvedValue({
       nodes: [
         {
           node_type: "Tweet",
           ref_id: "ref-1",
-          properties: { name: "Bitcoin is freedom", status: "complete", boost: 150, pubkey: "03abc123testkey" },
+          properties: {
+            name: "Bitcoin is freedom",
+            status: "complete",
+            boost: 150,
+            owner_reference_id: "lsat:11111111-1111-1111-1111-111111111111",
+          },
         },
       ],
       totalCount: 1,
       totalProcessing: 0,
     })
-    // pubKey matches node pubkey, no routeHint
     myContentUserOverrides = { pubKey: "03abc123testkey", routeHint: "", isAdmin: false }
     render(<MyContentPanel onClose={() => {}} />)
     await waitFor(() => {
@@ -191,7 +195,12 @@ describe("MyContentPanel", () => {
         {
           node_type: "Tweet",
           ref_id: "ref-1",
-          properties: { name: "Bitcoin is freedom", status: "complete", boost: 200, pubkey: "03someoneelse" },
+          properties: {
+            name: "Bitcoin is freedom",
+            status: "complete",
+            boost: 200,
+            owner_reference_id: "lsat:22222222-2222-2222-2222-222222222222",
+          },
         },
       ],
       totalCount: 1,
@@ -308,57 +317,35 @@ describe("MyContentPanel – delete button", () => {
     mockDeleteNode.mockResolvedValue({})
   })
 
-  const NODE_WITHOUT_UID = {
+  // /v2/content is server-filtered to the caller's own content, so every node here
+  // is implicitly owned — delete always renders, no client-side ownership check needed.
+  const NODE = {
     nodes: [
       {
         node_type: "Tweet",
         ref_id: "ref-orphan",
-        properties: { name: "Orphan Node", status: "error", pubkey: "03abc123testkey" },
-      },
-    ],
-    totalCount: 1,
-    totalProcessing: 0,
-  }
-
-  const NODE_OTHER_PUBKEY = {
-    nodes: [
-      {
-        node_type: "Tweet",
-        ref_id: "ref-other",
-        properties: { name: "Someone Else Node", status: "complete", pubkey: "03someoneelse" },
+        properties: {
+          name: "Orphan Node",
+          status: "error",
+          owner_reference_id: "lsat:11111111-1111-1111-1111-111111111111",
+        },
       },
     ],
     totalCount: 1,
     totalProcessing: 0,
   }
 
-  it("trash button renders when isAdmin = true", async () => {
-    myContentUserOverrides = { pubKey: "03abc123testkey", routeHint: "", isAdmin: true }
-    mockApiGet.mockResolvedValue(NODE_OTHER_PUBKEY)
-    render(<MyContentPanel onClose={() => {}} />)
-    await waitFor(() => expect(screen.getByText("Someone Else Node")).toBeInTheDocument())
-    expect(screen.getByRole("button", { name: /delete node/i })).toBeInTheDocument()
-  })
-
-  it("trash button renders when node pubkey matches user pubKey", async () => {
+  it("trash button always renders for /v2/content nodes", async () => {
     myContentUserOverrides = { pubKey: "03abc123testkey", routeHint: "", isAdmin: false }
-    mockApiGet.mockResolvedValue(NODE_WITHOUT_UID)
+    mockApiGet.mockResolvedValue(NODE)
     render(<MyContentPanel onClose={() => {}} />)
     await waitFor(() => expect(screen.getByText("Orphan Node")).toBeInTheDocument())
     expect(screen.getByRole("button", { name: /delete node/i })).toBeInTheDocument()
   })
 
-  it("trash button is absent when neither owner nor admin", async () => {
-    myContentUserOverrides = { pubKey: "03abc123testkey", routeHint: "", isAdmin: false }
-    mockApiGet.mockResolvedValue(NODE_OTHER_PUBKEY)
-    render(<MyContentPanel onClose={() => {}} />)
-    await waitFor(() => expect(screen.getByText("Someone Else Node")).toBeInTheDocument())
-    expect(screen.queryByRole("button", { name: /delete node/i })).toBeNull()
-  })
-
   it("clicking trash shows inline confirmation; Cancel hides it with no API call", async () => {
     myContentUserOverrides = { pubKey: "03abc123testkey", routeHint: "", isAdmin: false }
-    mockApiGet.mockResolvedValue(NODE_WITHOUT_UID)
+    mockApiGet.mockResolvedValue(NODE)
     render(<MyContentPanel onClose={() => {}} />)
     await waitFor(() => expect(screen.getByText("Orphan Node")).toBeInTheDocument())
 
@@ -373,7 +360,7 @@ describe("MyContentPanel – delete button", () => {
 
   it("confirm calls deleteNode(ref_id) and removes only that node", async () => {
     myContentUserOverrides = { pubKey: "03abc123testkey", routeHint: "", isAdmin: false }
-    mockApiGet.mockResolvedValue(NODE_WITHOUT_UID)
+    mockApiGet.mockResolvedValue(NODE)
     render(<MyContentPanel onClose={() => {}} />)
     await waitFor(() => expect(screen.getByText("Orphan Node")).toBeInTheDocument())