Fix linter issues for v2.0.0 release

Fixed 6 linter issues found in CI:

1. errcheck (2 issues):
   - internal/reader/reader.go:58 - Added proper Close() error handling with defer
   - internal/reader/reader.go:174 - Added proper Close() error handling with defer

2. gocritic (2 issues):
   - internal/reader/element.go:56 - Replaced single-case switch with if statement
   - internal/reader/reconstruct.go:1306 - Replaced single-case switch with if statement

3. staticcheck (1 issue):
   - internal/core/image.go:141 - Simplified with unconditional strings.TrimPrefix

4. unparam (1 issue):
   - internal/reader/reconstruct.go:414 - Changed hydrateRun() to return only error
     (domain.Run return value was never used)

All tests pass, 0 linter warnings.
Ready for production release v2.0.0.

Author: mmonterroca

internal/core/image.go

@@ -137,10 +137,7 @@ func NewImageFromPackage(target string, data []byte, contentType string) (domain
 		normalized = strings.TrimPrefix(normalized, "../")
 	}
 
-	relative := normalized
-	if strings.HasPrefix(relative, "word/") {
-		relative = strings.TrimPrefix(relative, "word/")
-	}
+	relative := strings.TrimPrefix(normalized, "word/")
 
 	base := filepath.Base(relative)
 	if base == "" {

internal/reader/element.go

@@ -53,8 +53,7 @@ func parseXMLTree(data []byte) (*Element, error) {
 		if err != nil {
 			return nil, err
 		}
-		switch start := tok.(type) {
-		case xml.StartElement:
+		if start, ok := tok.(xml.StartElement); ok {
 			return parseElement(dec, start)
 		}
 	}

internal/reader/reader.go

@@ -55,7 +55,9 @@ func LoadPackageFromPath(path string) (*Package, error) {
 	if err != nil {
 		return nil, errors.WrapWithCode(err, errors.ErrCodeIO, opLoadFromPath)
 	}
-	defer file.Close()
+	defer func() {
+		_ = file.Close() // Best effort close
+	}()
 
 	info, err := file.Stat()
 	if err != nil {
@@ -171,7 +173,9 @@ func readZipFile(file *zip.File) ([]byte, error) {
 	if err != nil {
 		return nil, err
 	}
-	defer rc.Close()
+	defer func() {
+		_ = rc.Close() // Best effort close
+	}()
 
 	data, err := io.ReadAll(rc)
 	if err != nil {

internal/reader/reconstruct.go

@@ -176,7 +176,7 @@ func populateParagraph(para domain.Paragraph, elem *Element, ctx *reconstructCon
 
 		switch child.Name.Local {
 		case "r":
-			if _, err := hydrateRun(para, child, ctx, state, nil); err != nil {
+			if err := hydrateRun(para, child, ctx, state, nil); err != nil {
 				return err
 			}
 		case "hyperlink":
@@ -411,9 +411,9 @@ func applyParagraphNumbering(para domain.Paragraph, props *Element) error {
 	return nil
 }
 
-func hydrateRun(para domain.Paragraph, elem *Element, ctx *reconstructContext, state *fieldState, extraFields []domain.Field) (domain.Run, error) {
+func hydrateRun(para domain.Paragraph, elem *Element, ctx *reconstructContext, state *fieldState, extraFields []domain.Field) error {
 	if para == nil || elem == nil {
-		return nil, nil
+		return nil
 	}
 
 	var (
@@ -438,7 +438,7 @@ func hydrateRun(para domain.Paragraph, elem *Element, ctx *reconstructContext, s
 		case "fldChar":
 			if state != nil {
 				if err := state.handleFieldChar(child); err != nil {
-					return nil, err
+					return err
 				}
 			}
 		case "instrText":
@@ -458,35 +458,35 @@ func hydrateRun(para domain.Paragraph, elem *Element, ctx *reconstructContext, s
 	}
 
 	if !createRun {
-		return nil, nil
+		return nil
 	}
 
 	run, err := para.AddRun()
 	if err != nil {
-		return nil, errors.Wrap(err, opHydrateRun)
+		return errors.Wrap(err, opHydrateRun)
 	}
 
 	if textBuilder.Len() > 0 {
 		if err := run.SetText(textBuilder.String()); err != nil {
-			return nil, errors.Wrap(err, opHydrateRun)
+			return errors.Wrap(err, opHydrateRun)
 		}
 	}
 
 	if props != nil {
 		if err := applyRunProperties(run, props); err != nil {
-			return nil, err
+			return err
 		}
 	}
 
 	for _, br := range breaks {
 		if err := run.AddBreak(br); err != nil {
-			return nil, errors.Wrap(err, opHydrateRun)
+			return errors.Wrap(err, opHydrateRun)
 		}
 	}
 
 	if state != nil {
 		if err := state.attachToRun(run); err != nil {
-			return nil, err
+			return err
 		}
 	}
 
@@ -501,19 +501,19 @@ func hydrateRun(para domain.Paragraph, elem *Element, ctx *reconstructContext, s
 			accessor.SetProperty("display", run.Text())
 		}
 		if err := run.AddField(field); err != nil {
-			return nil, errors.Wrap(err, opHydrateRun)
+			return errors.Wrap(err, opHydrateRun)
 		}
 	}
 
 	if len(drawings) > 0 {
 		for _, drawing := range drawings {
 			if err := hydrateDrawing(para, run, drawing, ctx); err != nil {
-				return nil, err
+				return err
 			}
 		}
 	}
 
-	return run, nil
+	return nil
 }
 
 func applyRunProperties(run domain.Run, props *Element) error {
@@ -681,7 +681,7 @@ func hydrateHyperlink(para domain.Paragraph, elem *Element, ctx *reconstructCont
 			extraFields = []domain.Field{field}
 		}
 
-		if _, err := hydrateRun(para, child, ctx, state, extraFields); err != nil {
+		if err := hydrateRun(para, child, ctx, state, extraFields); err != nil {
 			return errors.Wrap(err, opHydrateHyperlink)
 		}
 	}
@@ -720,7 +720,7 @@ func hydrateSimpleField(para domain.Paragraph, elem *Element, ctx *reconstructCo
 			}
 		}
 
-		if _, err := hydrateRun(para, child, ctx, state, extra); err != nil {
+		if err := hydrateRun(para, child, ctx, state, extra); err != nil {
 			return errors.Wrap(err, opHydrateSimpleField)
 		}
 	}
@@ -1303,8 +1303,7 @@ func extractQuotedStrings(input string) []string {
 	start := 0
 
 	for i, r := range input {
-		switch r {
-		case '"':
+		if r == '"' {
 			if inQuote {
 				results = append(results, input[start:i])
 				inQuote = false