Handle output PDF file directory location better

o Better paper format handling for cards, paper wallets
o Multiple Seeds enables Extra Entropy collection

Author: pjkundert

README.org

@@ -32,16 +32,16 @@ recovery phrases.  This is better, but creating such accounts is difficult; pres
 Trezor supports these, and they can only be created "manually".  Writing down 5 or more sets of 20
 words is difficult, error-prone and time consuming.
 
-The [[https://github.com/pjkundert/python-slip39.git][python-slip39]] project exists to assist in the safe creation and documentation of [[https://wolovim.medium.com/ethereum-201-hd-wallets-11d0c93c87][Ethereum HD
-Wallet]] seeds and derived accounts, with various SLIP-39 sharing parameters.  It generates the new
-random wallet seed, and generates the expected standard Ethereum account(s) (at [[https://medium.com/myetherwallet/hd-wallets-and-derivation-paths-explained-865a643c7bf2][derivation path]]
-=m/44'/60'/0'/0/0= by default) and Bitcoin accounts (at derivation path =m/84'/0'/0'/0/0= by
-default), with wallet address and QR code (compatible with Trezor derivations).  It produces the
-required SLIP-39 phrases, and outputs a single PDF containing all the required printable cards to
-document the seed (and the specified derived accounts).
+The [[https://github.com/pjkundert/python-slip39.git][python-slip39]] project (and the [[https://slip39.kundert.ca/macos][SLIP-39 macOS App]]) exists to assist in the safe creation and
+documentation of [[https://wolovim.medium.com/ethereum-201-hd-wallets-11d0c93c87][Ethereum HD Wallet]] seeds and derived accounts, with various SLIP-39 sharing
+parameters.  It generates the new random wallet seed, and generates the expected standard Ethereum
+account(s) (at [[https://medium.com/myetherwallet/hd-wallets-and-derivation-paths-explained-865a643c7bf2][derivation path]] =m/44'/60'/0'/0/0= by default) and Bitcoin accounts (at Bech32
+derivation path =m/84'/0'/0'/0/0= by default), with wallet address and QR code (compatible with
+Trezor derivations).  It produces the required SLIP-39 phrases, and outputs a single PDF containing
+all the required printable cards to document the seed (and the specified derived accounts).
 
-Output of BIP-38 or JSON encrypted paper wallets is supported, to support software cryptocurrency
-wallets.
+Output of BIP-38 or JSON encrypted Paper Wallets is supported, for import into standard software
+cryptocurrency wallets.
 
 On an secure (ideally air-gapped) computer, new seeds can safely be generated and the PDF saved to a
 USB drive for printing (or directly printed without the file being saved to disk.).  Presently,
@@ -54,16 +54,17 @@ by entering the mnemonics right on the device.
 * Security with Availability
 
   For both BIP-39 and SLIP-39, a 128-bit random "seed" is the source of an unlimited sequence of
-  Ethereum HD Wallet accounts.  Anyone who can obtain this seed gains control of all Ethereum,
-  Bitcoin (and other) accounts derived from it, so it must be securely stored.
+  Ethereum and Bitcoin HD (Heirarchical Deterministic) derived Wallet accounts.  Anyone who can
+  obtain this seed gains control of all Ethereum, Bitcoin (and other) accounts derived from it, so
+  it must be securely stored.
 
-  Losing this seed means that all of the HD Wallet accounts are permanently lost.  Therefore, it
-  must be backed up reliably, and be readily accessible.
+  Losing this seed means that all of the HD Wallet accounts are permanently lost.  It must be /both/
+  backed up securely, /and/ be readily accessible.
 
   Therefore, we must:
 
   - Ensure that nobody untrustworthy can recover the seed, but
-  - Store the seed in many places with several (some perhaps untrustworthy) people.
+  - Store the seed in many places, probably with several (some perhaps untrustworthy) people.
 
   How can we address these conflicting requirements?
 

README.pdf

@@ -116,15 +116,15 @@ of the cards and regain access to all of the cryptocurrency accounts related to
 ** Protecting your SLIP-39 Cards
 
    Protect your printed SLIP-39 cards from water damage by laminating them in plastic or storing
-   them in [[https://amzn.to/346xBU8][zip-loc bags]] before [[https://amzn.to/35m4][mailing them]].
+   them in [[https://amzn.to/346xBU8][zip-loc bags]] before [[https://amzn.to/3HCX8lv][mailing them]].
    #+BEGIN_EXPORT html
    <iframe style="width:120px;height:240px;" marginwidth="0" marginheight="0" scrolling="no" frameborder="0" src="//rcm-na.amazon-adsystem.com/e/cm?lt1=_blank&bc1=000000&IS2=1&bg1=FFFFFF&fc1=000000&lc1=0000FF&t=pjkundert-20&language=en_CA&o=15&p=8&l=as4&m=amazon&f=ifr&ref=as_ss_li_til&asins=B07X9GWPH1&linkId=69a3c4648067d4268b83e20be281ebb4"></iframe>
    <iframe style="width:120px;height:240px;" marginwidth="0" marginheight="0" scrolling="no" frameborder="0" src="//rcm-na.amazon-adsystem.com/e/cm?lt1=_blank&bc1=000000&IS2=1&bg1=FFFFFF&fc1=000000&lc1=0000FF&t=pjkundert-20&language=en_CA&o=15&p=8&l=as4&m=amazon&f=ifr&ref=as_ss_li_til&asins=B07WXMYX87&linkId=06e0f5d889c93f5427c379ddc5fa6857"></iframe>
    #+END_EXPORT
 
    Print the SLIP-39 cards and cut them out, and then lay them out with 1/2" margins (so you can cut
    them out after lamination and retain 1/4" borders), either with [[https://amzn.to/3K6wp2p][self-adhesive full-page
-   laminating sheets - no laminating machine required]] (or [[https://amzn.to/3vyyKPw][index-card size]]),
+   laminating sheets]] - no machine required (or [[https://amzn.to/3vyyKPw][index-card size]]),
    #+BEGIN_EXPORT html
    <iframe style="width:120px;height:240px;" marginwidth="0" marginheight="0" scrolling="no" frameborder="0" src="//rcm-na.amazon-adsystem.com/e/cm?lt1=_blank&bc1=000000&IS2=1&bg1=FFFFFF&fc1=000000&lc1=0000FF&t=pjkundert-20&language=en_CA&o=15&p=8&l=as4&m=amazon&f=ifr&ref=as_ss_li_til&asins=B00007E7D2&linkId=608ce5dd44a7a227327c9000d6442c92"></iframe>
    <iframe style="width:120px;height:240px;" marginwidth="0" marginheight="0" scrolling="no" frameborder="0" src="//rcm-na.amazon-adsystem.com/e/cm?lt1=_blank&bc1=000000&IS2=1&bg1=FFFFFF&fc1=000000&lc1=0000FF&t=pjkundert-20&language=en_CA&o=15&p=8&l=as4&m=amazon&f=ifr&ref=as_ss_li_til&asins=B00ENFRAX8&linkId=4ef3861c37b523826fcf6d3a87349890"></iframe>

README.txt

@@ -146,18 +146,17 @@ Table of Contents
   them].
   Print the SLIP-39 cards and cut them out, and then lay them out with
   1/2" margins (so you can cut them out after lamination and retain 1/4"
-  borders), either with [self-adhesive full-page laminating sheets - no
-  laminating machine required] (or [index-card size]),
+  borders), either with [self-adhesive full-page laminating sheets] - no
+  machine required (or [index-card size]),
   or with a [heat-laminating machine] in [full-page pouches] (or in
   [index-card size pouches]).
 
 
 [zip-loc bags] <https://amzn.to/346xBU8>
 
-[mailing them] <https://amzn.to/35m4>
+[mailing them] <https://amzn.to/3HCX8lv>
 
-[self-adhesive full-page laminating sheets - no laminating machine
-required] <https://amzn.to/3K6wp2p>
+[self-adhesive full-page laminating sheets] <https://amzn.to/3K6wp2p>
 
 [index-card size] <https://amzn.to/3vyyKPw>
 

macOS.org

@@ -71,6 +71,9 @@
 required SLIP-39 phrases, and outputs a single PDF containing all the required printable cards to
 document the seed (and the specified derived accounts).
 
+Output of BIP-38 or JSON encrypted Paper Wallets is supported, for import into standard software
+cryptocurrency wallets.
+
 On an secure (ideally air-gapped) computer, new seeds can safely be generated and the PDF saved to a
 USB drive for printing (or directly printed without the file being saved to disk.).  Presently,
 =slip39= can output example ETH, BTC, LTC and DOGE addresses derived from the seed, to illustrate

macOS.pdf

@@ -53,12 +53,12 @@
 CARD				= 'index'
 CARD_SIZES			= dict(
     index	= INDEX_CARD,
-    photo	= PHOTO_CARD,
     credit	= CREDIT_CARD,
     business	= BUSINESS_CARD,
     half	= HALF_LETTER,
     third	= THIRD_LETTER,
     quarter	= THIRD_LETTER,
+    photo	= PHOTO_CARD,
 )
 
 # Paper Wallet Bill Sizes, by default on PAPER format paper

macOS.txt

@@ -114,25 +114,21 @@ def groups_layout(
 
     layout                      = [
         [
-            sg.Frame( '1. Location for output PDF File(s) (Preferably removable media, such as a USB drive)', [
-                [
-                    sg.Text( "Save PDF(s) to: ",                                size=prefix,    **T_kwds ),
-                    sg.Input( sg.user_settings_get_entry( "-target folder-", ""),
-                                                        key='-TARGET-',         size=inputs,    **I_kwds ),  # noqa: E127
-                    sg.FolderBrowse( **B_kwds ),
-                    sg.Text( "Card size: ",                                                     **T_kwds ),
-                ] + [
-                    sg.Radio( f"{cs}", "CS",            key=f"-CS-{cs}", default=(cs == CARD),  **B_kwds )
-                    for cs in CARD_SIZES
-                ],
+            sg.Frame( '1. Name(s) and formats of SLIP-39 Cards for output PDF File(s)', [
                 [
                     sg.Text( "Seed Name(s): ",                                  size=prefix,    **T_kwds ),
                     sg.Input( f"{', '.join( names )}",  key='-NAMES-',          size=inputs,    **I_kwds ),
-                    sg.Text( ", ...   Paper size: ",                                **T_kwds ),
+                    sg.Text( ", ... on Paper: ",                                                **T_kwds ),
                 ] + [
-                    sg.Radio( f"{pf}", "PF",            key=f"-PF-{pf}", default=(pf == PAPER), **B_kwds )
+                    sg.Radio( f"{pf}", "PF",            key=f"-PF-{pf}-", default=(pf == PAPER), **B_kwds )
                     for pf in PAPER_FORMATS
                 ],
+                [
+                    sg.Text( "SLIP-39 Card size: ",                                             **T_kwds ),
+                ] + [
+                    sg.Radio( f"{cs}", "CS",            key=f"-CS-{cs}-", default=(cs == CARD), **B_kwds )
+                    for cs in CARD_SIZES
+                ],
             ],                                                  key='-OUTPUT-F-',               **F_kwds ),
         ],
     ] + [
@@ -270,7 +266,9 @@ def groups_layout(
         ],
     ] + [
         [
-            sg.Button( 'Save',                          key='-SAVE-',                           **B_kwds ),
+            sg.Input( sg.user_settings_get_entry( "-target folder-", ""),
+                                                        key='-SAVE-',           visible=False,  **I_kwds ),  # noqa: E127
+            sg.FolderBrowse( 'Save',                    target='-SAVE-',                        **B_kwds ),
             sg.Button( 'Exit',                                                                  **B_kwds ),
             sg.Frame( '6. Summary of your SLIP-39 Recovery Groups.  When this looks good, hit Save!', [
                 [
@@ -720,11 +718,11 @@ def app(
     groups_recovered		= None		# The last SLIP-39 groups recovered from user input
     details			= None		# ..and the SLIP-39 details produced; make None to force SLIP-39 Mnemonic update
     cryptopaths			= None
+    timeout			= 0		# First time thru; refresh immediately
     while event not in events_termination:
         # Create window (for initial window.read()), or update status
         if window:
-            window['-STATUS-'].update( f"{status or 'OK':.145}{'...' if len(status or 'OK')>145 else ''}", font=font_bold if status_error else font )
-            window['-SAVE-'].update( disabled=status_error )
+            window['-STATUS-'].update( f"{status or 'OK'}", font=font_bold if status_error else font )
             window['-RECOVERY-'].update( f"of {len(groups)}" )
             window['-SD-SEED-F-'].expand( expand_x=True )
             window['-SE-SEED-F-'].expand( expand_x=True )
@@ -736,14 +734,14 @@ def app(
             window['-RECOVERED-F-'].expand( expand_x=True )
             window['-MNEMONICS-F-'].expand( expand_x=True )
             window['-GROUPS-F-'].expand( expand_x=True )
-            timeout		= None 		# Subsequently, block indefinitely
         else:
             window		= sg.Window( f"{', '.join( names or [ 'SLIP-39' ] )} Mnemonic Cards", layout )
-            timeout		= 0 		# First time through, refresh immediately
+            timeout		= 0 		# First time through w/ new window, refresh immediately
 
         # Block (except for first loop) and obtain current event and input values. Until we get a
         # new event, retain the current status
         event, values		= window.read( timeout=timeout )
+        timeout			= None 		# Subsequently, default; block indefinitely
         logging.debug( f"{event}, {values}" )
         if not values or event in events_termination or event in events_ignored:
             continue
@@ -765,6 +763,23 @@ def app(
             window.extend_layout( window['-GROUP-NEEDS-'],  [[ sg.Input( f"{needs[0]}", key=f"-G-NEED-{g}", size=shorty, **I_kwds ) ]] )  # noqa: 241
             window.extend_layout( window['-GROUP-SIZES-'],  [[ sg.Input( f"{needs[1]}", key=f"-G-SIZE-{g}", size=shorty, **I_kwds ) ]] )  # noqa: 241
 
+        # Deduce the desired Seed names, defaulting to "SLIP39"; if any change, force update of
+        # details.  If the user specifies more than one, ensure that Extra Seed Entropy is enabled.
+        names_now		= [
+            name.strip()
+            for name in ( values['-NAMES-'].strip() or "SLIP39" ).split( ',' )
+            if name and name.strip()
+        ]
+        if names != names_now:
+            names		= names_now
+            details		= None
+            if names and len( names ) > 1:
+                if values['-SE-NON-']:
+                    # Multiple names specified, but NO Extra Seed Entropy selected!
+                    window['-SE-SHA-'].update( True )
+                    timeout	= 0
+                    continue
+
         # Attempt to compute the 1st Master Secret Seed, collecting any failure status detected.
         # Compute the Master Secret Seed, from the supplied Seed Data and any extra Seed Entropy.
         # We are displaying the 1st extra Seed Entropy, used to produce the first Seed, for the
@@ -910,16 +925,6 @@ def app(
 
         window['-SUMMARY-'].update( summary )
 
-        # Deduce the desired Seed names, defaulting to "SLIP39"; if any change, force update
-        names_now		= [
-            name.strip()
-            for name in ( values['-NAMES-'].strip() or "SLIP39" ).split( ',' )
-            if name and name.strip()
-        ]
-        if names != names_now:
-            names		= names_now
-            details		= None
-
         # Re-compute the SLIP39 Seed details.  For multiple names, each subsequent slip39.create
         # uses a master_secret produced by hashing the prior master_secret entropy.  For the 1st
         # Seed, we've computed the master_secret, above.  Each master_secret_n contains the computed
@@ -962,43 +967,38 @@ def app(
         # If all has gone well -- display the resultant <name>/<filename>, and some derived account details
         name_len		= max( len( name ) for name in details )
         status			= '\n'.join(
-            f"{name:>{name_len}} == " + ', '.join( f'{a.crypto} @ {a.path}: {a.address}' for a in details[name].accounts[0] )
+            f"{name:>{name_len}} == {', '.join( f'{a.crypto} @ {a.path}: {a.address}' for a in details[name].accounts[0] ):.130}..."
             for name in details
         )
 
-        # We have a complete SLIP-39 Mnemonic set.  A target directory must be selected; use it, if
-        # possible.  This is where any output will be written.  It should usually be a removable
-        # volume, but we do not check for this.
-        try:
-            os.chdir( values['-TARGET-'] )
-        except Exception as exc:
-            status		= f"Error changing to target directory {values['-TARGET-']!r}: {exc}"
-            logging.exception( f"{status}" )
-            update_seed_recovered( window, values, None )
-            continue
-
-        # If we get here, no failure status has been detected, and SLIP39 mnemonic and account
-        # details { "name": <details> } have been created; we can now save the PDFs; converted
-        # details is now { "<filename>": <details> })
+        # We have a complete SLIP-39 Mnemonic set.  If we get here, no failure status has been
+        # detected, and SLIP39 mnemonic and account details { "name": <details> } have been created;
+        # we can now save the PDFs; converted details is now { "<filename>": <details> })
         if event == '-SAVE-':
+            # A -SAVE- target directory has been selected; use it, if possible.  This is where any
+            # output will be written.  It should usually be a removable volume, but we do not check.
+            # An empty path implies the current directory.
             try:
-                card_format	= next( c for c in CARD_SIZES if values[f"-CS-{c}"] )
+                card_format	= next( c for c in CARD_SIZES if values[f"-CS-{c}-"] )
+                paper_format	= next( pf for pn,pf in PAPER_FORMATS.items() if values[f"-PF-{pn}-"] )
                 details		= write_pdfs(
                     names		= details,
                     card_format		= card_format,
+                    paper_format	= paper_format,
                     cryptocurrency	= cryptocurrency,
                     edit		= edit,
                     wallet_pwd		= values['-WALLET-PASS-'],  # Produces Paper Wallet(s) iff set
                     wallet_pwd_hint	= values['-WALLET-HINT-'],
                     wallet_format	= next( (f for f in WALLET_SIZES if values.get( f"-WALLET-SIZE-{f}-" )), None ),
+                    filepath		= values['-SAVE-'] or None,
                 )
             except Exception as exc:
                 status		= f"Error saving PDF(s): {exc}"
                 logging.exception( f"{status}" )
                 continue
             name_len		= max( len( name ) for name in details )
             status		= '\n'.join(
-                f"{name:>{name_len}} saved to {values['-TARGET-']}"
+                f"Saved {name}"
                 for name in details
             )
         # Finally, success has been assured; turn off emboldened status line