Commit 83a2897ed6c400b3679a37920e964d829094bd43 | the-knife

author	Julius Marminge <julius0216@outlook.com> 2026-09-05 01:20:50-07:00
committer	GitHub <noreply@github.com> 2026-09-05 01:20:50-07:00
log	83a2897ed6c400b3679a37920e964d829094bd43
tree	fad5007216e764092babcc8ba33f12da29538fc7
parent	68aa7aa8305e9dfc0d98543f607d88bd37fb66e2
signature	Signed by PGP key B5690EEEBB952194

refactor(mobile): test composer persistence through the live decoder (#10033)

2 files changed, 8 insertions(+), 13 deletions(-)

apps
- mobile
  - src
    state
    use-composer-drafts.test.ts
    use-composer-drafts.ts

apps/mobile/src/state/use-composer-drafts.test.ts+8-9

 @@ -159,7 +159,6 @@ import {
   copyComposerDraftContentIfEmpty,
   copyComposerDraftContentState,
   decodePersistedComposerState,
-  decodePersistedComposerDrafts,
   ensureComposerDraftsLoaded,
   type ComposerDraft,
   flushComposerDrafts,
 @@ -252,12 +251,12 @@ describe("mobile composer drafts", () => {
     };
     expect(
-      decodePersistedComposerDrafts({
+      decodePersistedComposerState({
         schemaVersion: 1,
         drafts: {
           "environment-1:thread-1": { text: "Review this file", attachments: [file] },
         },
-      }),
+      }).drafts,
     ).toEqual({
       "environment-1:thread-1": { text: "Review this file", attachments: [file] },
     });
 @@ -987,7 +986,7 @@ describe("mobile composer drafts", () => {
   it("rejects persisted images without image bytes or a file URI", () => {
     expect(() =>
-      decodePersistedComposerDrafts({
+      decodePersistedComposerState({
         schemaVersion: 1,
         drafts: {
           "environment-1:thread-1": {
 @@ -1010,7 +1009,7 @@ describe("mobile composer drafts", () => {
   it("hydrates selector state even when the message content is empty", () => {
     expect(
-      decodePersistedComposerDrafts({
+      decodePersistedComposerState({
         schemaVersion: 1,
         drafts: {
           "new-task:environment-1:project-1": {
 @@ -1030,7 +1029,7 @@ describe("mobile composer drafts", () => {
             },
           },
         },
-      }),
+      }).drafts,
     ).toEqual({
       "new-task:environment-1:project-1": {
         text: "",
 @@ -1053,18 +1052,18 @@ describe("mobile composer drafts", () => {
   it("keeps legacy content-only drafts and rejects invalid selector state", () => {
     expect(
-      decodePersistedComposerDrafts({
+      decodePersistedComposerState({
         schemaVersion: 1,
         drafts: {
           "environment-1:thread-1": DRAFT,
         },
-      }),
+      }).drafts,
     ).toEqual({
       "environment-1:thread-1": DRAFT,
     });
     expect(() =>
-      decodePersistedComposerDrafts({
+      decodePersistedComposerState({
         schemaVersion: 1,
         drafts: {
           "environment-1:thread-1": {

apps/mobile/src/state/use-composer-drafts.ts-4

 @@ -238,10 +238,6 @@ export function decodePersistedComposerState(value: unknown): {
   };
 }
-export function decodePersistedComposerDrafts(value: unknown): Record<string, ComposerDraft> {
-  return decodePersistedComposerState(value).drafts;
-}
 async function getComposerDraftsFile() {
   const { Directory, File, Paths } = await import("expo-file-system");
   const directory = new Directory(Paths.document, COMPOSER_DRAFTS_DIRECTORY);