canopycms 0.0.61-int.75 → 0.0.62

package/dist/editor/fields/ReferenceField.js.map

@@ -1 +1 @@
-{"version":3,"file":"ReferenceField.js","sourceRoot":"","sources":["../../../src/editor/fields/ReferenceField.tsx"],"names":[],"mappings":"AAAA,YAAY,CAAA;;AAEZ,OAAc,EAAE,SAAS,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,OAAO,CAAA;AAEzD,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,eAAe,CAAA;AACvF,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAA;AAErD,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAClD,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AAqBnD,MAAM,CAAC,MAAM,cAAc,GAAkC,CAAC,EAC5D,EAAE,EACF,KAAK,EACL,OAAO,EAAE,aAAa,EACtB,WAAW,EACX,UAAU,EACV,YAAY,GAAG,OAAO,EACtB,MAAM,GAAG,MAAM,EACf,KAAK,EACL,QAAQ,EACR,QAAQ,EACR,eAAe,GAChB,EAAE,EAAE;IACH,MAAM,cAAc,GAAG,CAAC,CAAC,WAAW,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,CAAA;IAC9D,MAAM,aAAa,GAAG,CAAC,CAAC,UAAU,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,CAAA;IAC3D,MAAM,UAAU,GAAG,CAAC,aAAa,IAAI,CAAC,cAAc,IAAI,aAAa,CAAC,CAAA;IACtE,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,GAAG,QAAQ,CAAoB,aAAa,IAAI,EAAE,CAAC,CAAA;IAC9E,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,GAAG,QAAQ,CAAC,UAAU,CAAC,CAAA;IAClD,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,QAAQ,CAAgB,IAAI,CAAC,CAAA;IACvD,MAAM,CAAC,UAAU,EAAE,aAAa,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAA;IAE/C,sEAAsE;IACtE,MAAM,SAAS,GAAG,CAAC,GAAY,EAAU,EAAE;QACzC,IAAI,OAAO,GAAG,KAAK,QAAQ;YAAE,OAAO,GAAG,CAAA;QACvC,IAAI,GAAG,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,IAAI,IAAI,GAAG,IAAI,OAAO,GAAG,CAAC,EAAE,KAAK,QAAQ;YAAE,OAAO,GAAG,CAAC,EAAE,CAAA;QAC9F,OAAO,EAAE,CAAA;IACX,CAAC,CAAA;IAED,MAAM,eAAe,GAAG,QAAQ;QAC9B,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;YACpB,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAC;YACtB,CAAC,CAAC,EAAE;QACN,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAA;IACpB,MAAM,WAAW,GAAG,KAAK,EAAE,CAAA;IAC3B,MAAM,OAAO,GAAG,EAAE,IAAI,WAAW,CAAA;IAEjC,+DAA+D;IAC/D,MAAM,CAAC,YAAY,EAAE,eAAe,CAAC,GAAG,QAAQ,CAAC,EAAE,CAAC,CAAA;IACpD,MAAM,QAAQ,GAAG,UAAU;QACzB,CAAC,CAAC,GAAG,WAAW,EAAE,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,UAAU,EAAE,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,YAAY,IAAI,MAAM,EAAE;QAC5F,CAAC,CAAC,EAAE,CAAA;IACN,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,eAAe,CAAC,QAAQ,CAAC,CAAA;QACzB,IAAI,UAAU,EAAE,CAAC;YACf,UAAU,CAAC,IAAI,CAAC,CAAA;YAChB,QAAQ,CAAC,IAAI,CAAC,CAAA;QAChB,CAAC;IACH,CAAC;IAED,wFAAwF;IACxF,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,UAAU,EAAE,CAAC;YACf,MAAM,SAAS,GAAG,eAAe,EAAE,CAAA;YAEnC,MAAM,MAAM,GAA2B,EAAE,MAAM,EAAE,YAAY,EAAE,CAAA;YAC/D,IAAI,WAAW,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC;gBAAE,MAAM,CAAC,WAAW,GAAG,WAAW,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;YACrF,IAAI,UAAU,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC;gBAAE,MAAM,CAAC,UAAU,GAAG,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;YAEjF,SAAS,CAAC,OAAO;iBACd,mBAAmB,CAAC,MAAM,CAAC;iBAC3B,IAAI,CAAC,CAAC,QAAQ,EAAE,EAAE;gBACjB,IAAI,QAAQ,CAAC,EAAE,IAAI,QAAQ,CAAC,IAAI,EAAE,OAAO,EAAE,CAAC;oBAC1C,MAAM,aAAa,GAAG,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,CAC7C,CAAC,GAAkC,EAAE,EAAE,CAAC,CAAC;wBACvC,KAAK,EAAE,GAAG,CAAC,EAAE;wBACb,KAAK,EAAE,GAAG,CAAC,KAAK;qBACjB,CAAC,CACH,CAAA;oBACD,UAAU,CAAC,aAAa,CAAC,CAAA;gBAC3B,CAAC;qBAAM,CAAC;oBACN,QAAQ,CAAC,QAAQ,CAAC,KAAK,IAAI,kCAAkC,CAAC,CAAA;gBAChE,CAAC;YACH,CAAC,CAAC;iBACD,KAAK,CAAC,CAAC,GAAG,EAAE,EAAE;gBACb,OAAO,CAAC,KAAK,CAAC,mCAAmC,EAAE,GAAG,CAAC,CAAA;gBACvD,QAAQ,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC,CAAA;YAChC,CAAC,CAAC;iBACD,OAAO,CAAC,GAAG,EAAE;gBACZ,UAAU,CAAC,KAAK,CAAC,CAAA;YACnB,CAAC,CAAC,CAAA;QACN,CAAC;IACH,CAAC,EAAE,CAAC,UAAU,EAAE,WAAW,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,EAAE,UAAU,CAAC,CAAC,CAAA;IAE3E,MAAM,WAAW,GAAG,GAAG,EAAE;QACvB,UAAU,CAAC,IAAI,CAAC,CAAA;QAChB,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,aAAa,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,CAAC,CAAA;IACrC,CAAC,CAAA;IAED,IAAI,OAAO,EAAE,CAAC;QACZ,OAAO,CACL,MAAC,KAAK,IACJ,GAAG,EAAE,CAAC,uBACa,eAAe,iBACrB,mBAAmB,eAAe,EAAE,aAEjD,KAAC,IAAI,IAAC,IAAI,EAAC,IAAI,EAAC,EAAE,EAAE,GAAG,YACpB,KAAK,GACD,EACP,KAAC,MAAM,IAAC,IAAI,EAAC,IAAI,iBAAc,qBAAqB,eAAe,EAAE,GAAI,IACnE,CACT,CAAA;IACH,CAAC;IAED,IAAI,KAAK,EAAE,CAAC;QACV,OAAO,CACL,MAAC,KAAK,IACJ,GAAG,EAAE,CAAC,uBACa,eAAe,iBACrB,mBAAmB,eAAe,EAAE,aAEjD,KAAC,IAAI,IAAC,IAAI,EAAC,IAAI,EAAC,EAAE,EAAE,GAAG,YACpB,KAAK,GACD,EACP,KAAC,KAAK,IACJ,IAAI,EAAE,KAAC,eAAe,IAAC,IAAI,EAAE,EAAE,GAAI,EACnC,KAAK,EAAC,KAAK,EACX,KAAK,EAAC,OAAO,iBACA,mBAAmB,eAAe,EAAE,YAEjD,MAAC,KAAK,IAAC,GAAG,EAAE,CAAC,aACX,KAAC,IAAI,IAAC,IAAI,EAAC,IAAI,YAAE,KAAK,GAAQ,EAC9B,KAAC,MAAM,IACL,IAAI,EAAC,IAAI,EACT,OAAO,EAAC,OAAO,EACf,OAAO,EAAE,WAAW,iBACP,mBAAmB,eAAe,EAAE,sBAG1C,IACH,GACF,IACF,CACT,CAAA;IACH,CAAC;IAED,OAAO,CACL,KAAC,KAAK,IACJ,GAAG,EAAE,CAAC,uBACa,eAAe,iBACrB,mBAAmB,eAAe,EAAE,YAEhD,QAAQ,CAAC,CAAC,CAAC,CACV,KAAC,WAAW,IACV,EAAE,EAAE,OAAO,EACX,KAAK,EAAE,KAAK,EACZ,IAAI,EAAE,OAAO,EACb,KAAK,EAAE,eAA2B,EAClC,QAAQ,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,EAClC,UAAU,QACV,WAAW,EAAC,wBAAmB,EAC/B,IAAI,EAAC,IAAI,GACT,CACH,CAAC,CAAC,CAAC,CACF,KAAC,MAAM,IACL,EAAE,EAAE,OAAO,EACX,KAAK,EAAE,KAAK,EACZ,IAAI,EAAE,OAAO,EACb,KAAK,EAAE,eAAyB,EAChC,QAAQ,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,IAAI,EAAE,CAAC,EACxC,UAAU,QACV,SAAS,QACT,WAAW,EAAC,wBAAmB,EAC/B,IAAI,EAAC,IAAI,GACT,CACH,GACK,CACT,CAAA;AACH,CAAC,CAAA;AAED,eAAe,cAAc,CAAA"}
+{"version":3,"file":"ReferenceField.js","sourceRoot":"","sources":["../../../src/editor/fields/ReferenceField.tsx"],"names":[],"mappings":"AAAA,YAAY,CAAA;;AAEZ,OAAc,EAAE,SAAS,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,OAAO,CAAA;AAEzD,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,eAAe,CAAA;AACvF,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAA;AAErD,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAClD,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AAqBnD,MAAM,CAAC,MAAM,cAAc,GAAkC,CAAC,EAC5D,EAAE,EACF,KAAK,EACL,OAAO,EAAE,aAAa,EACtB,WAAW,EACX,UAAU,EACV,YAAY,GAAG,OAAO,EACtB,MAAM,GAAG,MAAM,EACf,KAAK,EACL,QAAQ,EACR,QAAQ,EACR,eAAe,GAChB,EAAE,EAAE;IACH,MAAM,cAAc,GAAG,CAAC,CAAC,WAAW,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,CAAA;IAC9D,MAAM,aAAa,GAAG,CAAC,CAAC,UAAU,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,CAAA;IAC3D,MAAM,UAAU,GAAG,CAAC,aAAa,IAAI,CAAC,cAAc,IAAI,aAAa,CAAC,CAAA;IACtE,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,GAAG,QAAQ,CAAoB,aAAa,IAAI,EAAE,CAAC,CAAA;IAC9E,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,GAAG,QAAQ,CAAC,UAAU,CAAC,CAAA;IAClD,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,QAAQ,CAAgB,IAAI,CAAC,CAAA;IACvD,MAAM,CAAC,UAAU,EAAE,aAAa,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAA;IAE/C,sEAAsE;IACtE,MAAM,SAAS,GAAG,CAAC,GAAY,EAAU,EAAE;QACzC,IAAI,OAAO,GAAG,KAAK,QAAQ;YAAE,OAAO,GAAG,CAAA;QACvC,IAAI,GAAG,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,IAAI,IAAI,GAAG,IAAI,OAAO,GAAG,CAAC,EAAE,KAAK,QAAQ;YAAE,OAAO,GAAG,CAAC,EAAE,CAAA;QAC9F,OAAO,EAAE,CAAA;IACX,CAAC,CAAA;IAED,MAAM,eAAe,GAAG,QAAQ;QAC9B,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;YACpB,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAC;YACtB,CAAC,CAAC,EAAE;QACN,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAA;IACpB,MAAM,WAAW,GAAG,KAAK,EAAE,CAAA;IAC3B,MAAM,OAAO,GAAG,EAAE,IAAI,WAAW,CAAA;IAEjC,+DAA+D;IAC/D,MAAM,CAAC,YAAY,EAAE,eAAe,CAAC,GAAG,QAAQ,CAAC,EAAE,CAAC,CAAA;IACpD,MAAM,QAAQ,GAAG,UAAU;QACzB,CAAC,CAAC,GAAG,WAAW,EAAE,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,UAAU,EAAE,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,YAAY,IAAI,MAAM,EAAE;QAC5F,CAAC,CAAC,EAAE,CAAA;IACN,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,eAAe,CAAC,QAAQ,CAAC,CAAA;QACzB,IAAI,UAAU,EAAE,CAAC;YACf,UAAU,CAAC,IAAI,CAAC,CAAA;YAChB,QAAQ,CAAC,IAAI,CAAC,CAAA;QAChB,CAAC;IACH,CAAC;IAED,yFAAyF;IACzF,EAAE;IACF,0EAA0E;IAC1E,yEAAyE;IACzE,yEAAyE;IACzE,oEAAoE;IACpE,EAAE;IACF,yEAAyE;IACzE,wEAAwE;IACxE,uEAAuE;IACvE,QAAQ;IACR,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,CAAC,UAAU;YAAE,OAAM;QACvB,IAAI,MAAM,GAAG,IAAI,CAAA;QACjB,MAAM,SAAS,GAAG,eAAe,EAAE,CAAA;QAEnC,MAAM,MAAM,GAA2B,EAAE,MAAM,EAAE,YAAY,EAAE,CAAA;QAC/D,IAAI,WAAW,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC;YAAE,MAAM,CAAC,WAAW,GAAG,WAAW,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;QACrF,IAAI,UAAU,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC;YAAE,MAAM,CAAC,UAAU,GAAG,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;QAEjF,SAAS,CAAC,OAAO;aACd,mBAAmB,CAAC,MAAM,CAAC;aAC3B,IAAI,CAAC,CAAC,QAAQ,EAAE,EAAE;YACjB,IAAI,CAAC,MAAM;gBAAE,OAAM;YACnB,IAAI,QAAQ,CAAC,EAAE,IAAI,QAAQ,CAAC,IAAI,EAAE,OAAO,EAAE,CAAC;gBAC1C,MAAM,aAAa,GAAG,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAkC,EAAE,EAAE,CAAC,CAAC;oBACvF,KAAK,EAAE,GAAG,CAAC,EAAE;oBACb,KAAK,EAAE,GAAG,CAAC,KAAK;iBACjB,CAAC,CAAC,CAAA;gBACH,UAAU,CAAC,aAAa,CAAC,CAAA;YAC3B,CAAC;iBAAM,CAAC;gBACN,QAAQ,CAAC,QAAQ,CAAC,KAAK,IAAI,kCAAkC,CAAC,CAAA;YAChE,CAAC;QACH,CAAC,CAAC;aACD,KAAK,CAAC,CAAC,GAAG,EAAE,EAAE;YACb,IAAI,CAAC,MAAM;gBAAE,OAAM;YACnB,OAAO,CAAC,KAAK,CAAC,mCAAmC,EAAE,GAAG,CAAC,CAAA;YACvD,QAAQ,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC,CAAA;QAChC,CAAC,CAAC;aACD,OAAO,CAAC,GAAG,EAAE;YACZ,IAAI,CAAC,MAAM;gBAAE,OAAM;YACnB,UAAU,CAAC,KAAK,CAAC,CAAA;QACnB,CAAC,CAAC,CAAA;QAEJ,OAAO,GAAG,EAAE;YACV,MAAM,GAAG,KAAK,CAAA;QAChB,CAAC,CAAA;QACD,4MAA4M;IAC9M,CAAC,EAAE,CAAC,UAAU,EAAE,QAAQ,EAAE,UAAU,CAAC,CAAC,CAAA;IAEtC,MAAM,WAAW,GAAG,GAAG,EAAE;QACvB,UAAU,CAAC,IAAI,CAAC,CAAA;QAChB,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,aAAa,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,CAAC,CAAA;IACrC,CAAC,CAAA;IAED,IAAI,OAAO,EAAE,CAAC;QACZ,OAAO,CACL,MAAC,KAAK,IACJ,GAAG,EAAE,CAAC,uBACa,eAAe,iBACrB,mBAAmB,eAAe,EAAE,aAEjD,KAAC,IAAI,IAAC,IAAI,EAAC,IAAI,EAAC,EAAE,EAAE,GAAG,YACpB,KAAK,GACD,EACP,KAAC,MAAM,IAAC,IAAI,EAAC,IAAI,iBAAc,qBAAqB,eAAe,EAAE,GAAI,IACnE,CACT,CAAA;IACH,CAAC;IAED,IAAI,KAAK,EAAE,CAAC;QACV,OAAO,CACL,MAAC,KAAK,IACJ,GAAG,EAAE,CAAC,uBACa,eAAe,iBACrB,mBAAmB,eAAe,EAAE,aAEjD,KAAC,IAAI,IAAC,IAAI,EAAC,IAAI,EAAC,EAAE,EAAE,GAAG,YACpB,KAAK,GACD,EACP,KAAC,KAAK,IACJ,IAAI,EAAE,KAAC,eAAe,IAAC,IAAI,EAAE,EAAE,GAAI,EACnC,KAAK,EAAC,KAAK,EACX,KAAK,EAAC,OAAO,iBACA,mBAAmB,eAAe,EAAE,YAEjD,MAAC,KAAK,IAAC,GAAG,EAAE,CAAC,aACX,KAAC,IAAI,IAAC,IAAI,EAAC,IAAI,YAAE,KAAK,GAAQ,EAC9B,KAAC,MAAM,IACL,IAAI,EAAC,IAAI,EACT,OAAO,EAAC,OAAO,EACf,OAAO,EAAE,WAAW,iBACP,mBAAmB,eAAe,EAAE,sBAG1C,IACH,GACF,IACF,CACT,CAAA;IACH,CAAC;IAED,OAAO,CACL,KAAC,KAAK,IACJ,GAAG,EAAE,CAAC,uBACa,eAAe,iBACrB,mBAAmB,eAAe,EAAE,YAEhD,QAAQ,CAAC,CAAC,CAAC,CACV,KAAC,WAAW,IACV,EAAE,EAAE,OAAO,EACX,KAAK,EAAE,KAAK,EACZ,IAAI,EAAE,OAAO,EACb,KAAK,EAAE,eAA2B,EAClC,QAAQ,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,EAClC,UAAU,QACV,WAAW,EAAC,wBAAmB,EAC/B,IAAI,EAAC,IAAI,GACT,CACH,CAAC,CAAC,CAAC,CACF,KAAC,MAAM,IACL,EAAE,EAAE,OAAO,EACX,KAAK,EAAE,KAAK,EACZ,IAAI,EAAE,OAAO,EACb,KAAK,EAAE,eAAyB,EAChC,QAAQ,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,IAAI,EAAE,CAAC,EACxC,UAAU,QACV,SAAS,QACT,WAAW,EAAC,wBAAmB,EAC/B,IAAI,EAAC,IAAI,GACT,CACH,GACK,CACT,CAAA;AACH,CAAC,CAAA;AAED,eAAe,cAAc,CAAA"}

package/dist/editor/hooks/__test__/test-utils.d.ts

@@ -27,9 +27,25 @@ import { type MockApiClient } from '../../../api/__test__/mock-client';
  */
 export declare function setupMockApiClient(): Promise<MockApiClient>;
 /**
- * Create a wrapper component that provides the mock API client via context.
- *
- * Use this with renderHook to provide the ApiClientContext:
+ * Create a wrapper component that provides the mock API client via context,
+ * plus an SWR cache isolated to this wrapper instance.
+ *
+ * The isolated cache (`provider: () => new Map()`) matters: several hooks
+ * (useBranchManager, useEntryManager, useCommentSystem) now read through
+ * SWR-backed data hooks keyed by resource/branch (e.g. "canopy:branches",
+ * "canopy:entries:main"). Without a fresh cache per wrapper, tests across
+ * this file (and other hook test files in the same worker) would all share
+ * SWR's real global cache and collide on those keys -- a later test could
+ * see an earlier test's mocked response instead of its own. `dedupingInterval`
+ * matches the production `SWRProvider` (2000ms) rather than 0: the manager
+ * hooks' explicit reload functions (loadBranches, loadComments,
+ * refreshEntries) always issue a fresh, un-deduped fetch regardless of this
+ * value -- see their doc comments -- so a non-zero interval here doesn't
+ * risk masking a duplicate-request bug, and it's needed for dedup
+ * regression tests that mount under React.StrictMode to actually observe
+ * SWR's coalescing behavior.
+ *
+ * Use this with renderHook to provide both contexts:
  *
  * @example
  * ```ts
@@ -41,6 +57,25 @@ export declare function setupMockApiClient(): Promise<MockApiClient>;
 export declare function createApiClientWrapper(mockClient: MockApiClient): ({ children }: {
     children: React.ReactNode;
 }) => import("react/jsx-runtime").JSX.Element;
+/**
+ * Like `createApiClientWrapper`, but the tree is additionally wrapped in
+ * `React.StrictMode`, which double-invokes effects (mount -> cleanup ->
+ * remount) in dev. Use this for dedup regression tests: without SWR's
+ * request deduplication, each fetch-on-load hook (useBranchManager,
+ * useEntryManager, useCommentSystem) fired its request twice under Strict
+ * Mode -- exactly the duplicate-request bug this SWR migration fixes.
+ *
+ * @example
+ * ```ts
+ * const mockClient = await setupMockApiClient()
+ * const wrapper = createStrictModeApiClientWrapper(mockClient)
+ * const { result } = renderHook(() => useSomeHook(), { wrapper })
+ * // assert the mock fetch was called exactly once, not twice
+ * ```
+ */
+export declare function createStrictModeApiClientWrapper(mockClient: MockApiClient): ({ children }: {
+    children: React.ReactNode;
+}) => import("react/jsx-runtime").JSX.Element;
 /**
  * Setup mock window.location for tests that need to manipulate browser location.
  *

package/dist/editor/hooks/__test__/test-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"test-utils.d.ts","sourceRoot":"","sources":["../../../../src/editor/hooks/__test__/test-utils.tsx"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,MAAM,OAAO,CAAA;AACzB,OAAO,EAAE,EAAE,EAAE,KAAK,IAAI,EAAE,MAAM,QAAQ,CAAA;AACtC,OAAO,EAAuB,KAAK,aAAa,EAAE,MAAM,mCAAmC,CAAA;AAG3F;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,aAAa,CAAC,CAKjE;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,aAAa,IACtC,cAAc;IAAE,QAAQ,EAAE,KAAK,CAAC,SAAS,CAAA;CAAE,6CAGpE;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,GAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAO,cAYjF;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,IAAI,IAAI,CAIvC;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,gBAAgB,CAC9B,OAAO,GAAE,KAAK,CAAC,OAAO,GAAG,MAAM,GAAG,KAAK,GAAG,MAAM,GAAG,OAAO,CAAa,GAiBlE,MAAM,CAAC,OAAO,GAAG,MAAM,GAAG,KAAK,GAAG,MAAM,GAAG,OAAO,EAAE,UAAU,CAAC,OAAO,EAAE,CAAC,KAAK,CAAC,CAAC,GAAG;IACtF,OAAO,EAAE,MAAM,IAAI,CAAA;CACpB,CACF;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAsB,oBAAoB,CACxC,OAAO,GAAE;IACP,QAAQ,CAAC,EAAE;QAAE,IAAI,CAAC,EAAE,MAAM,CAAC;QAAC,MAAM,CAAC,EAAE,MAAM,CAAA;KAAE,CAAA;IAC7C,YAAY,CAAC,EAAE,OAAO,CAAA;CAClB;;;;GAcP"}
+{"version":3,"file":"test-utils.d.ts","sourceRoot":"","sources":["../../../../src/editor/hooks/__test__/test-utils.tsx"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,MAAM,OAAO,CAAA;AACzB,OAAO,EAAE,EAAE,EAAE,KAAK,IAAI,EAAE,MAAM,QAAQ,CAAA;AAEtC,OAAO,EAAuB,KAAK,aAAa,EAAE,MAAM,mCAAmC,CAAA;AAG3F;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,aAAa,CAAC,CAKjE;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,aAAa,IACtC,cAAc;IAAE,QAAQ,EAAE,KAAK,CAAC,SAAS,CAAA;CAAE,6CAOpE;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,gCAAgC,CAAC,UAAU,EAAE,aAAa,IAE1C,cAAc;IAAE,QAAQ,EAAE,KAAK,CAAC,SAAS,CAAA;CAAE,6CAO1E;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,GAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAO,cAYjF;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,IAAI,IAAI,CAIvC;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,gBAAgB,CAC9B,OAAO,GAAE,KAAK,CAAC,OAAO,GAAG,MAAM,GAAG,KAAK,GAAG,MAAM,GAAG,OAAO,CAAa,GAiBlE,MAAM,CAAC,OAAO,GAAG,MAAM,GAAG,KAAK,GAAG,MAAM,GAAG,OAAO,EAAE,UAAU,CAAC,OAAO,EAAE,CAAC,KAAK,CAAC,CAAC,GAAG;IACtF,OAAO,EAAE,MAAM,IAAI,CAAA;CACpB,CACF;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAsB,oBAAoB,CACxC,OAAO,GAAE;IACP,QAAQ,CAAC,EAAE;QAAE,IAAI,CAAC,EAAE,MAAM,CAAC;QAAC,MAAM,CAAC,EAAE,MAAM,CAAA;KAAE,CAAA;IAC7C,YAAY,CAAC,EAAE,OAAO,CAAA;CAClB;;;;GAcP"}

package/dist/editor/hooks/__test__/test-utils.js

@@ -1,5 +1,13 @@
 import { jsx as _jsx } from "react/jsx-runtime";
+/**
+ * Shared test utilities for hook tests.
+ *
+ * This file provides common setup patterns and helpers to reduce duplication
+ * across hook test files.
+ */
+import React from 'react';
 import { vi } from 'vitest';
+import { SWRConfig } from 'swr';
 import { createMockApiClient } from '../../../api/__test__/mock-client.js';
 import { ApiClientProvider } from '../../context/index.js';
 /**
@@ -27,9 +35,25 @@ export async function setupMockApiClient() {
     return mockClient;
 }
 /**
- * Create a wrapper component that provides the mock API client via context.
- *
- * Use this with renderHook to provide the ApiClientContext:
+ * Create a wrapper component that provides the mock API client via context,
+ * plus an SWR cache isolated to this wrapper instance.
+ *
+ * The isolated cache (`provider: () => new Map()`) matters: several hooks
+ * (useBranchManager, useEntryManager, useCommentSystem) now read through
+ * SWR-backed data hooks keyed by resource/branch (e.g. "canopy:branches",
+ * "canopy:entries:main"). Without a fresh cache per wrapper, tests across
+ * this file (and other hook test files in the same worker) would all share
+ * SWR's real global cache and collide on those keys -- a later test could
+ * see an earlier test's mocked response instead of its own. `dedupingInterval`
+ * matches the production `SWRProvider` (2000ms) rather than 0: the manager
+ * hooks' explicit reload functions (loadBranches, loadComments,
+ * refreshEntries) always issue a fresh, un-deduped fetch regardless of this
+ * value -- see their doc comments -- so a non-zero interval here doesn't
+ * risk masking a duplicate-request bug, and it's needed for dedup
+ * regression tests that mount under React.StrictMode to actually observe
+ * SWR's coalescing behavior.
+ *
+ * Use this with renderHook to provide both contexts:
  *
  * @example
  * ```ts
@@ -40,7 +64,29 @@ export async function setupMockApiClient() {
  */
 export function createApiClientWrapper(mockClient) {
     return function Wrapper({ children }) {
-        return _jsx(ApiClientProvider, { client: mockClient, children: children });
+        return (_jsx(SWRConfig, { value: { provider: () => new Map(), dedupingInterval: 2000 }, children: _jsx(ApiClientProvider, { client: mockClient, children: children }) }));
+    };
+}
+/**
+ * Like `createApiClientWrapper`, but the tree is additionally wrapped in
+ * `React.StrictMode`, which double-invokes effects (mount -> cleanup ->
+ * remount) in dev. Use this for dedup regression tests: without SWR's
+ * request deduplication, each fetch-on-load hook (useBranchManager,
+ * useEntryManager, useCommentSystem) fired its request twice under Strict
+ * Mode -- exactly the duplicate-request bug this SWR migration fixes.
+ *
+ * @example
+ * ```ts
+ * const mockClient = await setupMockApiClient()
+ * const wrapper = createStrictModeApiClientWrapper(mockClient)
+ * const { result } = renderHook(() => useSomeHook(), { wrapper })
+ * // assert the mock fetch was called exactly once, not twice
+ * ```
+ */
+export function createStrictModeApiClientWrapper(mockClient) {
+    const Inner = createApiClientWrapper(mockClient);
+    return function StrictWrapper({ children }) {
+        return (_jsx(React.StrictMode, { children: _jsx(Inner, { children: children }) }));
     };
 }
 /**

package/dist/editor/hooks/__test__/test-utils.js.map

@@ -1 +1 @@
-{"version":3,"file":"test-utils.js","sourceRoot":"","sources":["../../../../src/editor/hooks/__test__/test-utils.tsx"],"names":[],"mappings":";AAQA,OAAO,EAAE,EAAE,EAAa,MAAM,QAAQ,CAAA;AACtC,OAAO,EAAE,mBAAmB,EAAsB,MAAM,mCAAmC,CAAA;AAC3F,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAA;AAEjD;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB;IACtC,MAAM,EAAE,eAAe,EAAE,GAAG,MAAM,MAAM,CAAC,cAAc,CAAC,CAAA;IACxD,MAAM,UAAU,GAAG,mBAAmB,EAAE,CAAA;IACxC,EAAE,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,eAAe,CAAC,UAAiB,CAAC,CAAA;IAC7D,OAAO,UAAU,CAAA;AACnB,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,sBAAsB,CAAC,UAAyB;IAC9D,OAAO,SAAS,OAAO,CAAC,EAAE,QAAQ,EAAiC;QACjE,OAAO,KAAC,iBAAiB,IAAC,MAAM,EAAE,UAAiB,YAAG,QAAQ,GAAqB,CAAA;IACrF,CAAC,CAAA;AACH,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,iBAAiB,CAAC,UAA8C,EAAE;IAChF,MAAM,gBAAgB,GAAG,MAAM,CAAC,QAAoB,CAAA;IAEpD,OAAQ,MAAc,CAAC,QAAQ,CAC9B;IAAC,MAAc,CAAC,QAAQ,GAAG;QAC1B,IAAI,EAAE,OAAO,CAAC,IAAI,IAAI,mBAAmB;QACzC,MAAM,EAAE,OAAO,CAAC,MAAM,IAAI,EAAE;KAC7B,CAAA;IAED,OAAO,GAAG,EAAE;QACV,CAAC;QAAC,MAAc,CAAC,QAAQ,GAAG,gBAAgB,CAAA;IAC9C,CAAC,CAAA;AACH,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,gBAAgB;IAC9B,MAAM,gBAAgB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAA;IAChC,MAAM,CAAC,OAAO,CAAC,YAAY,GAAG,gBAAgB,CAAA;IAC9C,OAAO,gBAAgB,CAAA;AACzB,CAAC;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,UAAU,gBAAgB,CAC9B,UAA8D,CAAC,OAAO,CAAC;IAEvE,MAAM,KAAK,GAAQ,EAAE,CAAA;IAErB,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,KAAK,CAAC,MAAM,CAAC,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAA;IACxE,CAAC;IAED,MAAM,OAAO,GAAG,GAAG,EAAE;QACnB,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC;YACvC,CAAC;YAAC,GAAW,EAAE,WAAW,EAAE,CAAA;QAC9B,CAAC;IACH,CAAC,CAAA;IAED,OAAO;QACL,GAAG,KAAK;QACR,OAAO;KAGR,CAAA;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,UAGI,EAAE;IAEN,MAAM,UAAU,GAAG,MAAM,kBAAkB,EAAE,CAAA;IAE7C,MAAM,eAAe,GAAG,iBAAiB,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAA;IAC3D,MAAM,WAAW,GAAG,OAAO,CAAC,YAAY,KAAK,KAAK,CAAC,CAAC,CAAC,gBAAgB,EAAE,CAAC,CAAC,CAAC,SAAS,CAAA;IAEnF,OAAO;QACL,UAAU;QACV,WAAW;QACX,OAAO,EAAE,GAAG,EAAE;YACZ,eAAe,EAAE,CAAA;QACnB,CAAC;KACF,CAAA;AACH,CAAC"}
+{"version":3,"file":"test-utils.js","sourceRoot":"","sources":["../../../../src/editor/hooks/__test__/test-utils.tsx"],"names":[],"mappings":";AAAA;;;;;GAKG;AAEH,OAAO,KAAK,MAAM,OAAO,CAAA;AACzB,OAAO,EAAE,EAAE,EAAa,MAAM,QAAQ,CAAA;AACtC,OAAO,EAAE,SAAS,EAAE,MAAM,KAAK,CAAA;AAC/B,OAAO,EAAE,mBAAmB,EAAsB,MAAM,mCAAmC,CAAA;AAC3F,OAAO,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAA;AAEjD;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB;IACtC,MAAM,EAAE,eAAe,EAAE,GAAG,MAAM,MAAM,CAAC,cAAc,CAAC,CAAA;IACxD,MAAM,UAAU,GAAG,mBAAmB,EAAE,CAAA;IACxC,EAAE,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,eAAe,CAAC,UAAiB,CAAC,CAAA;IAC7D,OAAO,UAAU,CAAA;AACnB,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,MAAM,UAAU,sBAAsB,CAAC,UAAyB;IAC9D,OAAO,SAAS,OAAO,CAAC,EAAE,QAAQ,EAAiC;QACjE,OAAO,CACL,KAAC,SAAS,IAAC,KAAK,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE,CAAC,IAAI,GAAG,EAAE,EAAE,gBAAgB,EAAE,IAAI,EAAE,YACrE,KAAC,iBAAiB,IAAC,MAAM,EAAE,UAAiB,YAAG,QAAQ,GAAqB,GAClE,CACb,CAAA;IACH,CAAC,CAAA;AACH,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,gCAAgC,CAAC,UAAyB;IACxE,MAAM,KAAK,GAAG,sBAAsB,CAAC,UAAU,CAAC,CAAA;IAChD,OAAO,SAAS,aAAa,CAAC,EAAE,QAAQ,EAAiC;QACvE,OAAO,CACL,KAAC,KAAK,CAAC,UAAU,cACf,KAAC,KAAK,cAAE,QAAQ,GAAS,GACR,CACpB,CAAA;IACH,CAAC,CAAA;AACH,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,iBAAiB,CAAC,UAA8C,EAAE;IAChF,MAAM,gBAAgB,GAAG,MAAM,CAAC,QAAoB,CAAA;IAEpD,OAAQ,MAAc,CAAC,QAAQ,CAC9B;IAAC,MAAc,CAAC,QAAQ,GAAG;QAC1B,IAAI,EAAE,OAAO,CAAC,IAAI,IAAI,mBAAmB;QACzC,MAAM,EAAE,OAAO,CAAC,MAAM,IAAI,EAAE;KAC7B,CAAA;IAED,OAAO,GAAG,EAAE;QACV,CAAC;QAAC,MAAc,CAAC,QAAQ,GAAG,gBAAgB,CAAA;IAC9C,CAAC,CAAA;AACH,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,gBAAgB;IAC9B,MAAM,gBAAgB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAA;IAChC,MAAM,CAAC,OAAO,CAAC,YAAY,GAAG,gBAAgB,CAAA;IAC9C,OAAO,gBAAgB,CAAA;AACzB,CAAC;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,UAAU,gBAAgB,CAC9B,UAA8D,CAAC,OAAO,CAAC;IAEvE,MAAM,KAAK,GAAQ,EAAE,CAAA;IAErB,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,KAAK,CAAC,MAAM,CAAC,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAA;IACxE,CAAC;IAED,MAAM,OAAO,GAAG,GAAG,EAAE;QACnB,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC;YACvC,CAAC;YAAC,GAAW,EAAE,WAAW,EAAE,CAAA;QAC9B,CAAC;IACH,CAAC,CAAA;IAED,OAAO;QACL,GAAG,KAAK;QACR,OAAO;KAGR,CAAA;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,UAGI,EAAE;IAEN,MAAM,UAAU,GAAG,MAAM,kBAAkB,EAAE,CAAA;IAE7C,MAAM,eAAe,GAAG,iBAAiB,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAA;IAC3D,MAAM,WAAW,GAAG,OAAO,CAAC,YAAY,KAAK,KAAK,CAAC,CAAC,CAAC,gBAAgB,EAAE,CAAC,CAAC,CAAC,SAAS,CAAA;IAEnF,OAAO;QACL,UAAU;QACV,WAAW;QACX,OAAO,EAAE,GAAG,EAAE;YACZ,eAAe,EAAE,CAAA;QACnB,CAAC;KACF,CAAA;AACH,CAAC"}

package/dist/editor/hooks/index.d.ts

@@ -1,8 +1,11 @@
 export * from './useEditorLayout';
 export { useBranchManager, type UseBranchManagerOptions } from './useBranchManager';
-export { useEntryManager } from './useEntryManager';
+export { useEntryManager, listAllEntries } from './useEntryManager';
 export * from './useDraftManager';
 export { useCommentSystem } from './useCommentSystem';
+export { useBranchesData, fetchBranches, BRANCHES_KEY, type BranchesData } from './useBranchesData';
+export { fetchEntriesAndSchema, entriesKey, type EntriesData, type FetchEntriesParams, } from './useEntriesData';
+export { useCommentsData, fetchComments, commentsKey, type CommentsData } from './useCommentsData';
 export { useGroupManager } from './useGroupManager';
 export { usePermissionManager } from './usePermissionManager';
 export { useBranchActions } from './useBranchActions';

package/dist/editor/hooks/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/editor/hooks/index.ts"],"names":[],"mappings":"AAIA,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAE,KAAK,uBAAuB,EAAE,MAAM,oBAAoB,CAAA;AACnF,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AACnD,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AACnD,OAAO,EAAE,oBAAoB,EAAE,MAAM,wBAAwB,CAAA;AAC7D,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,cAAc,EAAE,KAAK,oBAAoB,EAAE,MAAM,kBAAkB,CAAA;AAC5E,OAAO,EACL,sBAAsB,EACtB,KAAK,6BAA6B,EAClC,KAAK,4BAA4B,GAClC,MAAM,0BAA0B,CAAA;AACjC,OAAO,EACL,gBAAgB,EAChB,KAAK,uBAAuB,EAC5B,KAAK,sBAAsB,GAC5B,MAAM,oBAAoB,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/editor/hooks/index.ts"],"names":[],"mappings":"AAIA,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAE,KAAK,uBAAuB,EAAE,MAAM,oBAAoB,CAAA;AACnF,OAAO,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAA;AACnE,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,YAAY,EAAE,KAAK,YAAY,EAAE,MAAM,mBAAmB,CAAA;AACnG,OAAO,EACL,qBAAqB,EACrB,UAAU,EACV,KAAK,WAAW,EAChB,KAAK,kBAAkB,GACxB,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,WAAW,EAAE,KAAK,YAAY,EAAE,MAAM,mBAAmB,CAAA;AAClG,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AACnD,OAAO,EAAE,oBAAoB,EAAE,MAAM,wBAAwB,CAAA;AAC7D,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,cAAc,EAAE,KAAK,oBAAoB,EAAE,MAAM,kBAAkB,CAAA;AAC5E,OAAO,EACL,sBAAsB,EACtB,KAAK,6BAA6B,EAClC,KAAK,4BAA4B,GAClC,MAAM,0BAA0B,CAAA;AACjC,OAAO,EACL,gBAAgB,EAChB,KAAK,uBAAuB,EAC5B,KAAK,sBAAsB,GAC5B,MAAM,oBAAoB,CAAA"}

package/dist/editor/hooks/index.js

@@ -3,9 +3,12 @@
 // Note: resetApiClient functions are not exported - they're test-only utilities
 export * from './useEditorLayout.js';
 export { useBranchManager } from './useBranchManager.js';
-export { useEntryManager } from './useEntryManager.js';
+export { useEntryManager, listAllEntries } from './useEntryManager.js';
 export * from './useDraftManager.js';
 export { useCommentSystem } from './useCommentSystem.js';
+export { useBranchesData, fetchBranches, BRANCHES_KEY } from './useBranchesData.js';
+export { fetchEntriesAndSchema, entriesKey, } from './useEntriesData.js';
+export { useCommentsData, fetchComments, commentsKey } from './useCommentsData.js';
 export { useGroupManager } from './useGroupManager.js';
 export { usePermissionManager } from './usePermissionManager.js';
 export { useBranchActions } from './useBranchActions.js';

package/dist/editor/hooks/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/editor/hooks/index.ts"],"names":[],"mappings":"AAAA,qCAAqC;AACrC,mCAAmC;AACnC,gFAAgF;AAEhF,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAgC,MAAM,oBAAoB,CAAA;AACnF,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AACnD,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AACnD,OAAO,EAAE,oBAAoB,EAAE,MAAM,wBAAwB,CAAA;AAC7D,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,cAAc,EAA6B,MAAM,kBAAkB,CAAA;AAC5E,OAAO,EACL,sBAAsB,GAGvB,MAAM,0BAA0B,CAAA;AACjC,OAAO,EACL,gBAAgB,GAGjB,MAAM,oBAAoB,CAAA"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/editor/hooks/index.ts"],"names":[],"mappings":"AAAA,qCAAqC;AACrC,mCAAmC;AACnC,gFAAgF;AAEhF,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAgC,MAAM,oBAAoB,CAAA;AACnF,OAAO,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAA;AACnE,cAAc,mBAAmB,CAAA;AACjC,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,YAAY,EAAqB,MAAM,mBAAmB,CAAA;AACnG,OAAO,EACL,qBAAqB,EACrB,UAAU,GAGX,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,WAAW,EAAqB,MAAM,mBAAmB,CAAA;AAClG,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAA;AACnD,OAAO,EAAE,oBAAoB,EAAE,MAAM,wBAAwB,CAAA;AAC7D,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACrD,OAAO,EAAE,cAAc,EAA6B,MAAM,kBAAkB,CAAA;AAC5E,OAAO,EACL,sBAAsB,GAGvB,MAAM,0BAA0B,CAAA;AACjC,OAAO,EACL,gBAAgB,GAGjB,MAAM,oBAAoB,CAAA"}

package/dist/editor/hooks/useBranchManager.d.ts

@@ -20,6 +20,8 @@ export interface BranchSummary {
     mergedAt?: string;
     /** Sync status for async GitHub operations (used when Lambda has no internet) */
     syncStatus?: SyncStatus;
+    /** Short, sanitized reason the last GitHub sync task failed (set alongside syncStatus: 'sync-failed') */
+    syncFailureReason?: string;
     /** Whether this branch has unresolved merge conflicts with the base branch */
     conflictStatus?: ConflictStatus;
     /** ContentIds of entries where --theirs was applied during rebase; cleared on clean rebase */
@@ -27,6 +29,25 @@ export interface BranchSummary {
     commentCount: number;
     isProtected: boolean;
     readOnly: boolean;
+    /**
+     * Server-computed: content writes are rejected, because the branch is the
+     * read-only base branch OR its status is past 'editing'. `readOnly` still
+     * says WHICH, for banner copy.
+     */
+    writeBlocked: boolean;
+    /**
+     * Server-computed compound submit rule: true when the branch can never be
+     * submitted for review, for EITHER reason -- it is the protected base
+     * branch, or its workflow status has moved past 'editing'. Mirrors
+     * `writeBlocked`'s shape (a base-branch half + a status half, combined),
+     * except this one is built from `isProtected`/`submitBlocked` (both modes)
+     * rather than `readOnly` (prod-only) -- see
+     * `BranchWriteProtection.submitBlockedIncludingStatus` for why those two
+     * compounds are genuinely different predicates, not duplicates of each
+     * other. Consumed as-is by `BranchManager.tsx`'s `canSubmit`; do not
+     * re-derive the status/protection halves client-side.
+     */
+    submitBlocked: boolean;
 }
 export interface UseBranchManagerOptions {
     /**
@@ -52,7 +73,6 @@ export interface UseBranchManagerReturn {
     branches: BranchListItem[];
     branchSummaries: BranchSummary[];
     currentBranch: BranchListItem | undefined;
-    branchStatus: string;
     handleSubmit: (branchName: string) => Promise<void>;
     handleWithdraw: (branchName: string) => Promise<void>;
     handleRequestChanges: (branchName: string) => Promise<void>;

package/dist/editor/hooks/useBranchManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useBranchManager.d.ts","sourceRoot":"","sources":["../../../src/editor/hooks/useBranchManager.tsx"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,cAAc,EAAE,gBAAgB,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAC/E,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAA;AACzD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAA;AACxD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAA;AAoDtD;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,MAAM,CAAA;IACd,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,EAAE;QACN,KAAK,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;QAC3B,MAAM,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;KAC7B,CAAA;IACD,cAAc,CAAC,EAAE,MAAM,CAAA;IACvB,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAC1B,gBAAgB,CAAC,EAAE,gBAAgB,CAAA;IACnC,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,iFAAiF;IACjF,UAAU,CAAC,EAAE,UAAU,CAAA;IACvB,8EAA8E;IAC9E,cAAc,CAAC,EAAE,cAAc,CAAA;IAC/B,8FAA8F;IAC9F,aAAa,CAAC,EAAE,MAAM,EAAE,CAAA;IACxB,YAAY,EAAE,MAAM,CAAA;IACpB,WAAW,EAAE,OAAO,CAAA;IACpB,QAAQ,EAAE,OAAO,CAAA;CAClB;AAED,MAAM,WAAW,uBAAuB;IACtC;;OAEG;IACH,aAAa,EAAE,MAAM,CAAA;IAErB;;OAEG;IACH,aAAa,EAAE,aAAa,CAAA;IAE5B;;OAEG;IACH,OAAO,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAA;IAEhC;;OAEG;IACH,QAAQ,EAAE,aAAa,EAAE,CAAA;CAC1B;AAED,MAAM,WAAW,sBAAsB;IACrC,UAAU,EAAE,MAAM,CAAA;IAClB,aAAa,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAA;IACrC,QAAQ,EAAE,cAAc,EAAE,CAAA;IAC1B,eAAe,EAAE,aAAa,EAAE,CAAA;IAChC,aAAa,EAAE,cAAc,GAAG,SAAS,CAAA;IACzC,YAAY,EAAE,MAAM,CAAA;IACpB,YAAY,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IACnD,cAAc,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IACrD,oBAAoB,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IAC3D,YAAY,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IACnD,sBAAsB,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAA;IAC3C,YAAY,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAA;CAClC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,uBAAuB,GAAG,sBAAsB,CA0NzF"}
+{"version":3,"file":"useBranchManager.d.ts","sourceRoot":"","sources":["../../../src/editor/hooks/useBranchManager.tsx"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,cAAc,EAAE,gBAAgB,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAC/E,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAA;AACzD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAA;AACxD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAA;AA0HtD;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,MAAM,CAAA;IACd,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,EAAE;QACN,KAAK,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;QAC3B,MAAM,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;KAC7B,CAAA;IACD,cAAc,CAAC,EAAE,MAAM,CAAA;IACvB,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAC1B,gBAAgB,CAAC,EAAE,gBAAgB,CAAA;IACnC,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,iFAAiF;IACjF,UAAU,CAAC,EAAE,UAAU,CAAA;IACvB,yGAAyG;IACzG,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAC1B,8EAA8E;IAC9E,cAAc,CAAC,EAAE,cAAc,CAAA;IAC/B,8FAA8F;IAC9F,aAAa,CAAC,EAAE,MAAM,EAAE,CAAA;IACxB,YAAY,EAAE,MAAM,CAAA;IACpB,WAAW,EAAE,OAAO,CAAA;IACpB,QAAQ,EAAE,OAAO,CAAA;IACjB;;;;OAIG;IACH,YAAY,EAAE,OAAO,CAAA;IACrB;;;;;;;;;;;OAWG;IACH,aAAa,EAAE,OAAO,CAAA;CACvB;AAED,MAAM,WAAW,uBAAuB;IACtC;;OAEG;IACH,aAAa,EAAE,MAAM,CAAA;IAErB;;OAEG;IACH,aAAa,EAAE,aAAa,CAAA;IAE5B;;OAEG;IACH,OAAO,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAA;IAEhC;;OAEG;IACH,QAAQ,EAAE,aAAa,EAAE,CAAA;CAC1B;AAED,MAAM,WAAW,sBAAsB;IACrC,UAAU,EAAE,MAAM,CAAA;IAClB,aAAa,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAA;IACrC,QAAQ,EAAE,cAAc,EAAE,CAAA;IAC1B,eAAe,EAAE,aAAa,EAAE,CAAA;IAChC,aAAa,EAAE,cAAc,GAAG,SAAS,CAAA;IACzC,YAAY,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IACnD,cAAc,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IACrD,oBAAoB,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IAC3D,YAAY,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;IACnD,sBAAsB,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAA;IAC3C,YAAY,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAA;CAClC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,uBAAuB,GAAG,sBAAsB,CA4RzF"}

package/dist/editor/hooks/useBranchManager.js

@@ -1,39 +1,104 @@
 'use client';
 import { jsx as _jsx } from "react/jsx-runtime";
 import { useEffect, useMemo, useState } from 'react';
+import { useSWRConfig } from 'swr';
 import { Text } from '@mantine/core';
 import { modals } from '@mantine/modals';
 import { notifications } from '@mantine/notifications';
 import { useApiClient } from '../context/index.js';
+import { BRANCHES_KEY, fetchBranches, useBranchesData } from './useBranchesData.js';
 // branch-name, NOT branch or the '../../paths' barrel: both of those pull
 // node:fs/promises + node:path at the top level (path RESOLUTION helpers),
 // which breaks adopters' production `next build` of the editor bundle.
 // paths/branch-name.ts is the dependency-free home of sanitizeBranchName.
 import { sanitizeBranchName } from '../../paths/branch-name.js';
+/**
+ * Wire a confirm modal's exits so exactly ONE of {confirmed, dismissed} runs,
+ * whichever happens first.
+ *
+ * Neither Mantine callback is the "user declined" signal on its own, in
+ * opposite directions: `onCancel` does NOT fire for an Escape or overlay
+ * dismissal (so a caller waiting on it alone waits forever), while `onClose`
+ * fires for EVERY exit including a confirm -- Mantine closes the modal right
+ * after calling `onConfirm`, and does not await it, so a bare `onClose`
+ * handler would settle the caller's promise before the confirmed work had
+ * even run. Claiming the first exit resolves both.
+ */
+const confirmModalHandlers = (onConfirm, onDismiss) => {
+    let settled = false;
+    const claim = () => {
+        if (settled)
+            return false;
+        settled = true;
+        return true;
+    };
+    return {
+        onConfirm: () => {
+            // Not awaited by Mantine either way; the handlers below own their own
+            // error reporting, so nothing here can reject.
+            if (claim())
+                void onConfirm();
+        },
+        onCancel: () => {
+            if (claim())
+                onDismiss();
+        },
+        onClose: () => {
+            if (claim())
+                onDismiss();
+        },
+    };
+};
 /**
  * Helper function to show confirmation modal for branch submit action.
  */
-const showSubmitConfirmation = (branchName, onConfirm, onCancel) => {
+const showSubmitConfirmation = (branchName, onConfirm, onDismiss) => {
     modals.openConfirmModal({
         title: 'Submit Branch for Review',
         children: (_jsx(Text, { size: "sm", style: { whiteSpace: 'pre-line' }, children: `Are you sure you want to submit "${branchName}" for review?\n\nThis will:\n• Create a pull request for review\n• Change the branch status to "submitted"\n• Notify reviewers of pending changes` })),
         labels: { confirm: 'Submit Branch', cancel: 'Cancel' },
         confirmProps: { color: 'brand' },
-        onCancel,
-        onConfirm,
+        ...confirmModalHandlers(onConfirm, onDismiss),
     });
 };
 /**
  * Helper function to show confirmation modal for branch withdraw action.
  */
-const showWithdrawConfirmation = (branchName, onConfirm, onCancel) => {
+const showWithdrawConfirmation = (branchName, onConfirm, onDismiss) => {
     modals.openConfirmModal({
         title: 'Withdraw Branch from Review',
         children: (_jsx(Text, { size: "sm", style: { whiteSpace: 'pre-line' }, children: `Are you sure you want to withdraw "${branchName}" from review?\n\nThis will:\n• Convert the pull request to a draft\n• Change the branch status back to "editing"\n• Remove from review queue` })),
         labels: { confirm: 'Withdraw Branch', cancel: 'Cancel' },
         confirmProps: { color: 'orange' },
-        onCancel,
-        onConfirm,
+        ...confirmModalHandlers(onConfirm, onDismiss),
+    });
+};
+/**
+ * Helper function to show confirmation modal for branch delete action.
+ *
+ * Unlike submit/withdraw (both reversible), delete is irreversible -- it
+ * unlinks branch.json, removes the clone, and removes the branch head from
+ * the local git mirror. Before this, `handleDelete` below called
+ * `apiClient.branches.delete` immediately on click with no confirmation
+ * anywhere in the chain (BranchManager.tsx's delete button calls `onDelete`
+ * directly), while submit and withdraw -- both reversible -- each went
+ * through this same `modals.openConfirmModal` pattern. This closes that gap;
+ * `color: 'red'` marks it as the destructive one, matching the button's own
+ * color in BranchManager.tsx.
+ */
+const showDeleteConfirmation = (branchName, onConfirm, onDismiss) => {
+    modals.openConfirmModal({
+        title: 'Delete Branch',
+        children: (_jsx(Text, { size: "sm", style: { whiteSpace: 'pre-line' }, children: `Are you sure you want to delete "${branchName}"?\n\nThis will:\n• Permanently remove the branch and its clone\n• Discard any unsaved or unmerged changes\n\nThis cannot be undone.` })),
+        labels: { confirm: 'Delete Branch', cancel: 'Cancel' },
+        // data-testid is load-bearing, not decoration: the e2e helper
+        // (apps/test-app/e2e/fixtures/branch-page.ts's deleteBranch) already
+        // anticipated a delete confirmation and clicks
+        // [data-testid="confirm-delete-branch"] when it appears. Without this the
+        // modal opens, nothing dismisses it, and the branch never disappears --
+        // which is exactly how the e2e suite caught this confirmation being added.
+        confirmProps: { color: 'red', 'data-testid': 'confirm-delete-branch' },
+        ...confirmModalHandlers(onConfirm, onDismiss),
     });
 };
 /**
@@ -73,15 +138,51 @@ const showWithdrawConfirmation = (branchName, onConfirm, onCancel) => {
  */
 export function useBranchManager(options) {
     const apiClient = useApiClient();
+    const { mutate: globalMutate } = useSWRConfig();
     const [branchName, setBranchName] = useState(options.initialBranch);
-    const [branches, setBranches] = useState([]);
+    // Automatic load, deduped by SWR (e.g. React Strict Mode's double effect
+    // invoke collapses to a single request). Not keyed by branchName -- the
+    // endpoint returns every branch regardless of which one is selected.
+    const { data: branchesData, error: branchesError, isValidating: branchesIsValidating, } = useBranchesData(apiClient);
+    const branches = useMemo(() => branchesData?.branches ?? [], [branchesData]);
+    // Adopt the server's default branch once data arrives, if nothing pinned one.
+    useEffect(() => {
+        if (!branchName && branchesData?.defaultBranch) {
+            setBranchName(branchesData.defaultBranch);
+        }
+    }, [branchesData, branchName]);
+    // Surface/clear the sticky error toast the same way loadBranches() used to.
+    useEffect(() => {
+        if (branchesError) {
+            console.error(branchesError);
+            const message = branchesError instanceof Error ? branchesError.message : 'Failed to load branches';
+            // Fixed id: retries update the existing toast instead of stacking; sticky
+            // because the editor cannot function without the branch list.
+            notifications.show({
+                id: 'canopy-branches-load-failed',
+                message,
+                color: 'red',
+                autoClose: false,
+            });
+        }
+        else if (branchesData) {
+            // A previous failure may have left the sticky error toast up; clear it
+            // now that loading succeeded (provisioning failures are often transient).
+            notifications.hide('canopy-branches-load-failed');
+        }
+    }, [branchesData, branchesError]);
+    // Mirrors the automatic load's in-flight state onto the shared busy flag,
+    // matching loadBranches()'s own setBusy bracket below for explicit reloads.
+    useEffect(() => {
+        options.setBusy(branchesIsValidating);
+        // eslint-disable-next-line react-hooks/exhaustive-deps -- options is a new object every Editor.tsx render; only setBusy (a stable setState) matters here
+    }, [branchesIsValidating, options.setBusy]);
     // Exact match first; fall back to comparing sanitized forms so a legacy
     // deep-link carrying the raw, unsanitized name (e.g. "?branch=feature%2Fx"
     // from before a branch was created, or from an old bookmark) still
     // resolves to the branch the server actually persisted (e.g. "feature-x").
     const currentBranch = branches.find((b) => b.name === branchName) ??
         branches.find((b) => b.name === sanitizeBranchName(branchName));
-    const branchStatus = currentBranch?.status ?? 'editing';
     // Compute branch summaries with comment counts
     const branchSummaries = useMemo(() => {
         return branches.map((b) => {
@@ -101,40 +202,44 @@ export function useBranchManager(options) {
                 pullRequestState: b.pullRequestState,
                 mergedAt: b.mergedAt,
                 syncStatus: b.syncStatus,
+                syncFailureReason: b.syncFailureReason,
                 conflictStatus: b.conflictStatus,
                 conflictFiles: b.conflictFiles,
                 commentCount: unresolvedCount,
-                isProtected: b.isProtected ?? false,
+                // Fail CLOSED (`?? true`): these flags are optional on the wire (see
+                // BranchListItem's doc comment in api/branch.ts) so a version-skewed
+                // server, or a branches-list fetch that returned partial/legacy data,
+                // can omit them. `isProtected` gates `canDelete`/`isSystemBranch` in
+                // BranchManager.tsx and `submitBlocked` gates `canSubmit` -- both real
+                // mutating actions -- so "no answer" must render as "protected"/
+                // "blocked", the same fail-closed direction as `writeBlocked` below
+                // (see Editor.tsx's `branchContentLocked` for the full rationale, which
+                // applies identically here).
+                isProtected: b.isProtected ?? true,
+                // NOT flipped to `?? true`: `readOnly` only selects WHICH lock banner
+                // to show (base-branch read-only vs. status lock) once something is
+                // already known to be locked via `writeBlocked`/`submitBlocked` --
+                // it never gates a write by itself. Defaulting it true on missing data
+                // would mislabel a status lock as a base-branch lock (wrong banner
+                // copy), not under-lock anything, so `?? false` stays correct here.
                 readOnly: b.readOnly ?? false,
+                writeBlocked: b.writeBlocked ?? true,
+                submitBlocked: b.submitBlocked ?? true,
             };
         });
     }, [branches, branchName, options.comments]);
+    // Explicit reload: always issues a fresh, independent fetch (raw call, not
+    // SWR's `mutate()` revalidate path) so callers requesting a reload right
+    // after mount aren't coalesced against the still-in-flight automatic load
+    // -- then writes the result into the shared cache so useBranchesData's
+    // bound hook (and anything else reading BRANCHES_KEY) picks it up.
+    // Side effects (default-branch adoption, error/success notifications) are
+    // driven by the effects above, which react to that same cache write.
     const loadBranches = async () => {
         options.setBusy(true);
         try {
-            const result = await apiClient.branches.list();
-            if (result.status === 404) {
-                // No branch endpoint available; stay branchless. The branch dropdown
-                // stays clickable so the user can open Manage Branches (which also
-                // retries this load) and create or select a branch from there.
-                setBranches([]);
-                return;
-            }
-            if (!result.ok) {
-                // Surface the server's reason (e.g. workspace provisioning failures
-                // now arrive as 503s with the underlying git error)
-                throw new Error(result.error ?? `Failed to load branches: ${result.status}`);
-            }
-            const list = result.data?.branches ?? [];
-            setBranches(list);
-            // A previous failure may have left the sticky error toast up; clear it
-            // now that loading succeeded (provisioning failures are often transient).
-            notifications.hide('canopy-branches-load-failed');
-            // No branch pinned via URL or client config — adopt the server's
-            // effective default (the detected active branch in dev mode).
-            if (!branchName && result.data?.defaultBranch) {
-                setBranchName(result.data.defaultBranch);
-            }
+            const fresh = await fetchBranches(apiClient);
+            await globalMutate(BRANCHES_KEY, fresh, { revalidate: false });
         }
         catch (err) {
             console.error(err);
@@ -178,7 +283,13 @@ export function useBranchManager(options) {
                 finally {
                     options.setBusy(false);
                 }
-            }, () => reject(new Error('User cancelled submit')));
+            }, 
+            // Dismissal is not an error. Rejecting here meant clicking Cancel
+            // logged a console error at the only call site (Editor.tsx catches
+            // these into console.error), which is both noise and a trip hazard
+            // for the CI=1 stray-console gate. Settled once, by onCancel OR
+            // onClose, so an Escape/overlay dismissal can't leave this pending.
+            () => resolve());
         });
     };
     const handleWithdraw = async (branchNameToWithdraw) => {
@@ -204,7 +315,13 @@ export function useBranchManager(options) {
                 finally {
                     options.setBusy(false);
                 }
-            }, () => reject(new Error('User cancelled withdraw')));
+            }, 
+            // Dismissal is not an error. Rejecting here meant clicking Cancel
+            // logged a console error at the only call site (Editor.tsx catches
+            // these into console.error), which is both noise and a trip hazard
+            // for the CI=1 stray-console gate. Settled once, by onCancel OR
+            // onClose, so an Escape/overlay dismissal can't leave this pending.
+            () => resolve());
         });
     };
     const handleRequestChanges = async (branchNameForChanges) => {
@@ -226,33 +343,42 @@ export function useBranchManager(options) {
         }
     };
     const handleDelete = async (branchNameToDelete) => {
-        options.setBusy(true);
-        try {
-            const result = await apiClient.branches.delete({
-                branch: branchNameToDelete,
-            });
-            if (!result.ok) {
-                throw new Error(result.error || 'Failed to delete branch');
-            }
-            notifications.show({ message: 'Branch deleted', color: 'green' });
-            await loadBranches();
-        }
-        catch (err) {
-            const message = err instanceof Error ? err.message : 'Failed to delete branch';
-            notifications.show({ message, color: 'red' });
-        }
-        finally {
-            options.setBusy(false);
-        }
+        return new Promise((resolve) => {
+            showDeleteConfirmation(branchNameToDelete, async () => {
+                options.setBusy(true);
+                try {
+                    const result = await apiClient.branches.delete({
+                        branch: branchNameToDelete,
+                    });
+                    if (!result.ok) {
+                        throw new Error(result.error || 'Failed to delete branch');
+                    }
+                    notifications.show({ message: 'Branch deleted', color: 'green' });
+                    await loadBranches();
+                }
+                catch (err) {
+                    const message = err instanceof Error ? err.message : 'Failed to delete branch';
+                    notifications.show({ message, color: 'red' });
+                }
+                finally {
+                    options.setBusy(false);
+                    // Resolve regardless of API success/failure, matching the
+                    // pre-confirmation behavior (errors were shown as a toast, not
+                    // rethrown). Cancellation resolves too -- see below.
+                    resolve();
+                }
+            }, 
+            // Dismissal is not an error. Rejecting here meant clicking Cancel
+            // logged a console error at the only call site (Editor.tsx catches
+            // these into console.error), which is both noise and a trip hazard
+            // for the CI=1 stray-console gate. Settled once, by onCancel OR
+            // onClose, so an Escape/overlay dismissal can't leave this pending.
+            () => resolve());
+        });
     };
     const handleReloadBranchData = async () => {
         await loadBranches();
     };
-    // Load branches on mount and when branchName changes
-    useEffect(() => {
-        loadBranches().catch(console.error);
-        // eslint-disable-next-line react-hooks/exhaustive-deps -- loadBranches would cause infinite loop
-    }, [branchName]);
     // Sync branch to URL parameter
     useEffect(() => {
         if (typeof window === 'undefined')
@@ -272,7 +398,6 @@ export function useBranchManager(options) {
         branches,
         branchSummaries,
         currentBranch,
-        branchStatus,
         handleSubmit,
         handleWithdraw,
         handleRequestChanges,