Proposed solution for PR cuthbertLab#1636. Note that I had to check md['software'] as it was being constructed, instead of afterward, because Metadata() actually adds a software item for the music21 version that read the file, and I don't want to be confused by that.

Author: gregchapman-dev

music21/_version.py

@@ -50,7 +50,7 @@
 '''
 from __future__ import annotations
 
-__version__ = '9.6.0b4'
+__version__ = '9.6.0b19'
 
 def get_version_tuple(vv):
     v = vv.split('.')

music21/base.py

@@ -27,7 +27,7 @@
 <class 'music21.base.Music21Object'>
 
 >>> music21.VERSION_STR
-'9.6.0b4'
+'9.6.0b19'
 
 Alternatively, after doing a complete import, these classes are available
 under the module "base":

music21/musicxml/xmlToM21.py

@@ -769,6 +769,7 @@ def __init__(self):
         self.parts = []
 
         self.musicXmlVersion = defaults.musicxmlVersion
+        self.wasWrittenByFinale = False
 
     def scoreFromFile(self, filename):
         '''
@@ -1326,9 +1327,17 @@ def processEncoding(self, encoding: ET.Element, md: metadata.Metadata) -> None:
         # TODO: encoder (text + type = role) multiple
         # TODO: encoding date multiple
         # TODO: encoding-description (string) multiple
+        finaleFound: bool = False
+        nonFinaleFound: bool = False
         for software in encoding.findall('software'):
             if softwareText := strippedText(software):
+                if 'Finale' in softwareText:
+                    finaleFound = True
+                else:
+                    nonFinaleFound = True
                 md.add('software', softwareText)
+        if finaleFound and not nonFinaleFound:
+            self.wasWrittenByFinale = True
 
         for supports in encoding.findall('supports'):
             # todo: element: required
@@ -2630,18 +2639,19 @@ def xmlForward(self, mxObj: ET.Element):
         if durationText := strippedText(mxDuration):
             change = opFrac(float(durationText) / self.divisions)
 
-            # Create hidden rest (in other words, a spacer)
-            # old Finale documents close incomplete final measures with <forward>
-            # this will be removed afterward by removeEndForwardRest()
-            r = note.Rest(quarterLength=change)
-            r.style.hideObjectOnPrint = True
-            self.addToStaffReference(mxObj, r)
-            self.insertInMeasureOrVoice(mxObj, r)
+            if self.parent.parent.wasWrittenByFinale:
+                # Create hidden rest (in other words, a spacer)
+                # old Finale documents close incomplete final measures with <forward>
+                # this will be removed afterward by removeEndForwardRest()
+                r = note.Rest(quarterLength=change)
+                r.style.hideObjectOnPrint = True
+                self.addToStaffReference(mxObj, r)
+                self.insertInMeasureOrVoice(mxObj, r)
+                # xmlToNote() sets None
+                self.endedWithForwardTag = r
 
             # Allow overfilled measures for now -- TODO(someday): warn?
             self.offsetMeasureNote += change
-            # xmlToNote() sets None
-            self.endedWithForwardTag = r
 
     def xmlPrint(self, mxPrint: ET.Element):
         '''