Studio: Address comments from 2271

[katinthehatsite]

Related issues

Fixes:

• Studio: Add UI for when local tree for push has errors #2271 (review)
• Studio: Add UI for when local tree for push has errors #2271 (comment)

Proposed Changes

This PR addresses misc comments from #2271 that were submitted after the PR was merged.

Testing Instructions

• Pull the changes from this branch
• Apply the following diff:

index 31db0fc9..b538c66b 100644
--- a/src/stores/sync/sync-hooks.ts
+++ b/src/stores/sync/sync-hooks.ts
@@ -83,6 +83,9 @@ export function useLocalFileTree() {
                        setIsLoading( true );
                        setError( null );
                        try {
+                               // TEMPORARY: Simulate error for testing
+                               throw new Error( 'Simulated error for testing error handling' );
+                               
                                const rawNodes = await getIpcApi().listLocalFileTree( siteId, path, 3 );
                                return convertRawToTreeNodes( rawNodes );
                        } catch ( err ) {

• Start Studio with npm start
• Ensure that you have at least one WP.com site connected to Studio
• Navigate to the Sync tab
• Click on Push
• Select Specific files and folders option
• Confirm that when you open wp-content, you see the error shown on the screenshot above

Pre-merge Checklist

• Have you checked for TypeScript, React or other console errors?

[gavande1]

LGTM 👍

[katinthehatsite]

Thanks for the review @gavande1 ❤️