Successfully built SLIP-39.app

o Simplify Backup display
o Correctly deduce user name

Author: pjkundert

GNUmakefile

@@ -613,13 +613,13 @@ dist/SLIP-39.app: 		SLIP-39-macOS.spec \
 	#     $@
 	# echo "Checking signature (app code signed)..."; ./SLIP-39.metadata/check-signature $@ || true
 	# codesign --verify $@
-	# codesign --deep --force \
-	#     --all-architectures --options=runtime --timestamp \
-	#     --entitlements ./SLIP-39.metadata/entitlements.plist \
-	#     --sign "$(DEVID)" \
-	#     $@
-	# echo "Checking signature (app code + entitlements signed w/ $(DEVID))..."; ./SLIP-39.metadata/check-signature $@ || true
-	# codesign --verify $@
+	codesign --deep --force \
+	    --all-architectures --options=runtime --timestamp \
+	    --entitlements ./SLIP-39.metadata/entitlements.plist \
+	    --sign "$(DEVID)" \
+	    $@
+	echo "Checking signature (app code + entitlements signed w/ $(DEVID))..."; ./SLIP-39.metadata/check-signature $@ || true
+	codesign --verify $@
 	touch $@  # try to avoid unnecessary rebuilding
 
 #

README.org

@@ -86,7 +86,7 @@ slip39 -q --secret "seven replace great luggage fox rent general tower guess ins
 
    #+RESULTS:
    :results:
-   SLIP39-2023-10-26+13.21.19-ETH-0x6E6268F14B922cb924C7683A415B30C2bf967000.pdf
+   SLIP39-2023-10-27+05.09.52-ETH-0x6E6268F14B922cb924C7683A415B30C2bf967000.pdf
    :end:
 
    #+LATEX: }
@@ -109,18 +109,19 @@ slip39 -q --secret "seven replace great luggage fox rent general tower guess ins
    [[./images/SLIP39-recover-BIP39-entropy.png]]
 
    Later, when you need to recover your BIP-39 Seed Entropy and Mnemonic, use this SLIP-39 App or
-   [[https://iancoleman.com]], and entry some of your SLIP-39 Mnemonic Cards.  These may need to be collected from friends and family.
+   [[https://iancoleman.com/slip39/]] and entry some of your SLIP-39 Mnemonic Cards.  These may need
+   to be collected from friends and family.
 
    In this case, we're using the First and Second cards, intended for you to secure, separately from
-   each other; for example, in two safes or other secure locations like locked filing cabinets, and
+   each other; for example, in two safes or other secure locations like locked filing cabinets, at
    2 locations known to you and your partner(s):
 
    #+CAPTION: BIP-39 Mnemonic
    #+ATTR_LATEX: :float wrap :width 2in :placement {r}
    [[./images/SLIP39-recover-BIP39-mnemonic.png]]
 
    Finally, convert the recovered Seed Entropy back to your BIP-39 Mnemonic.  This requires 2 steps
-   if you use [[https://iancoleman.com]].
+   if you use [[https://iancoleman.com/bip39/]]
 
    In this step, we're simply converting the recovered Seed Entropy back into its BIP-39 Mnemonic.
    You need to select the "[X] show entropy details" checkbox in order to enter the raw Seed Entropy

README.pdf

@@ -13,7 +13,7 @@ a = Analysis(['SLIP-39.py'],
              pathex=[],
              binaries=[],
              datas=datas,
-             hiddenimports=[],
+             hiddenimports=['ipaddress'],
              hookspath=[],
              hooksconfig={},
              runtime_hooks=[],
@@ -50,9 +50,9 @@ coll = COLLECT(exe,
 app = BUNDLE(coll,
              name='SLIP-39.app',
              icon='images/SLIP-39.icns',
-             version='9.1.2',
+             version='11.0.2',
              info_plist={
-                 'CFBundleVersion':'9.1.2',
+                 'CFBundleVersion':'11.0.2',
                  'CFBundlePackageType':'APPL',
                  'LSApplicationCategoryType':'public.app-category.finance',
                  'LSMinimumSystemVersion':'10.15.0',

README.txt

@@ -2,7 +2,7 @@ aiosmtpd		>=1.4,	<2
 build
 cx_Freeze		>=6.12
 flake8
-pyinstaller		>=5.5
+pyinstaller		>=6.1
 pytest			>=7.4.2,<8
 pytest-cov		>=4.1.0,<5
 setuptools

SLIP-39-macOS.spec

@@ -34,7 +34,7 @@
 
 from ..api		import Account, create, group_parser, random_secret, cryptopaths_parser, paper_wallet_available
 from ..recovery		import recover, recover_bip39, produce_bip39, scan_entropy, display_entropy
-from ..util		import log_level, log_cfg, ordinal, commas, chunker, hue_shift, rate_dB, entropy_rating_dB, timing, avg
+from ..util		import log_level, log_cfg, ordinal, commas, chunker, hue_shift, rate_dB, entropy_rating_dB, timing, avg, parse_scutil
 from ..layout		import write_pdfs, printers_available
 from ..defaults		import (
     GROUPS, GROUP_THRESHOLD_RATIO, MNEM_PREFIX, CRYPTO_PATHS, BITS, BITS_BIP39, BITS_DEFAULT,
@@ -277,11 +277,10 @@ def groups_layout(
                                                                                 visible=LO_CRE, **T_hue( B_kwds, 0/20 )),
                     sg.Radio( "256-bit",          "SD", key='-SD-256-RND-',     visible=LO_CRE, **T_hue( B_kwds, 0/20 )),
                     sg.Radio( "512-bit",          "SD", key='-SD-512-RND-',     visible=LO_PRO, **T_hue( B_kwds, 0/20 )),
-                    sg.Text( "Recover:",                                        visible=LO_BAK, **T_hue( T_kwds, 2/20 )),
+                    sg.Text( "Recover:",                                        visible=LO_CRE, **T_hue( T_kwds, 2/20 )),
                     sg.Radio( "SLIP-39",          "SD", key='-SD-SLIP-',        visible=LO_REC, **T_hue( B_kwds, 2/20 )),
                     sg.Radio( "BIP-39",           "SD", key='-SD-BIP-',         default=LO_BAK,
-                                                                                visible=LO_BAK or LO_CRE,
-                                                                                                **T_hue( B_kwds, 2/20 )),
+                                                                                visible=LO_CRE, **T_hue( B_kwds, 2/20 )),
                     sg.Radio( "BIP-39 Seed",      "SD", key='-SD-BIP-SEED-',    visible=LO_PRO, **T_hue( B_kwds, 2/20 )),
                     sg.Checkbox( 'Passphrase',          key='-SD-PASS-C-',      visible=False,  **T_hue( B_kwds, 2/20 )),
                 ],
@@ -350,9 +349,10 @@ def groups_layout(
                     sg.Text( "",                        key='-SEED-',           size=inlong,    **T_kwds ),
                 ],
                 [
-                    sg.Checkbox( "Using BIP-39:",       key='-AS-BIP-CB-',      default=True,
+                    sg.Checkbox( "Using BIP-39:",       key='-AS-BIP-CB-',      visible=LO_CRE,
+                                                                                default=True,
                                                                                 size=prefix,    **T_hue( B_kwds, -1/20 )),
-                    sg.Text( '',                        key='-AS-BIP-',         visible=True,   **T_hue( T_kwds_dense, -1/20 )),
+                    sg.Text( '',                        key='-AS-BIP-',         visible=LO_CRE, **T_hue( T_kwds_dense, -1/20 )),
                 ],
                 [
                     sg.Column( [
@@ -602,12 +602,12 @@ def update_seed_data( event, window, values ):
         # We're recovering the BIP-39 Seed Phrase *Entropy*, NOT the derived (decrypted) 512-bit
         # Seed Data!  So, we don't deal in Passphrases, here.  The Passphrase (to encrypt the Seed,
         # when "Using BIP-39") is only required to display the correct wallet addresses.
-        window['-SD-DATA-F-'].update( "BIP-39 Mnemonic: " )
+        window['-SD-DATA-F-'].update( "BIP-39 Mnemonic to Back Up: " )
         window['-SD-DATA-F-'].update( visible=True )
         window['-SD-PASS-C-'].update( visible=False )
         window['-SD-PASS-F-'].update( visible=False )
     elif 'SLIP' in update_seed_data.src:
-        window['-SD-DATA-F-'].update( "SLIP-39 Mnemonics: " )
+        window['-SD-DATA-F-'].update( "SLIP-39 Mnemonics to Back Up: " )
         window['-SD-DATA-F-'].update( visible=True )
         window['-SD-PASS-C-'].update( visible=True )
         window['-SD-PASS-F-'].update(
@@ -1026,11 +1026,13 @@ def user_name_full():
         f"{' '.join( command )!r} command failed, or no output returned"
 
     if sys.platform == 'darwin':
-        for li in subproc.stdout.split( '\n' ):
-            if 'kCGSessionLongUserNameKey' in li:
-                # eg.: "      kCGSessionLongUserNameKey : Perry Kundert"
-                full_name	= li.split( ':' )[1].strip()
-                break
+        scutil = parse_scutil( subproc.stdout )
+        if uid := scutil.get( 'UID' ):
+            for session in scutil.get( 'SessionInfo', {} ).values():
+                if session.get( 'kCGSSessionUserIDKey' ) == uid:
+                    # eg.: "      kCGSessionLongUserNameKey : Perry Kundert"
+                    full_name = session.get( 'kCGSessionLongUserNameKey' )
+                    break
     elif sys.platform == 'win32':
         for li in subproc.stdout.split( '\n' ):
             if li.startswith( 'Full Name' ):
@@ -1352,7 +1354,9 @@ def deficiency( *deficiencies ):
 
         # Recover any passphrase, discarding any details on change.  The empty passphrase b'' is the
         # default for both SLIP-39 and BIP-39.
-        window['-AS-BIP-'].update( visible=values['-AS-BIP-CB-'] )
+        if window['-AS-BIP-CB-'].visible:
+            # Only if the checkbox is visible in this mode, also make the BIP-39 rendering visible
+            window['-AS-BIP-'].update( visible=values['-AS-BIP-CB-'] )
         window['-PASSPHRASE-F-'].update(
             'BIP-39 Passphrase' if using_bip39 else passphrase_trezor_incompatible )
         passphrase_now		= values['-PASSPHRASE-'].strip().encode( 'UTF-8' )