fix: remove unused functions from crawler (U1000)

- Remove extractLinksWithErr and extractFormsWithErr - dead code
  with no callers in the codebase

All tests pass with -race flag.

Author: Patel230

internal/crawler/parse.go

@@ -37,21 +37,6 @@ func extractLinks(pageURL string, body []byte) []Link {
 	return extractLinksFromDoc(pageURL, doc)
 }
 
-// extractLinksWithErr extracts links and reports any parse error via errOut.
-func extractLinksWithErr(pageURL string, body []byte, errOut *error) []Link {
-	doc, err := html.Parse(bytes.NewReader(body))
-	if err != nil {
-		if errOut != nil {
-			*errOut = err
-		}
-		// Still attempt extraction from whatever was parsed
-		if doc == nil {
-			return nil
-		}
-	}
-	return extractLinksFromDoc(pageURL, doc)
-}
-
 // extractLinksFromDoc extracts links from a pre-parsed HTML document.
 func extractLinksFromDoc(pageURL string, doc *html.Node) []Link {
 	if doc == nil {
@@ -174,20 +159,6 @@ func extractForms(body []byte) []Form {
 	return extractFormsFromDoc(doc)
 }
 
-// extractFormsWithErr extracts forms and reports any parse error via errOut.
-func extractFormsWithErr(body []byte, errOut *error) []Form {
-	doc, err := html.Parse(bytes.NewReader(body))
-	if err != nil {
-		if errOut != nil {
-			*errOut = err
-		}
-		if doc == nil {
-			return nil
-		}
-	}
-	return extractFormsFromDoc(doc)
-}
-
 // extractFormsFromDoc extracts forms from a pre-parsed HTML document.
 func extractFormsFromDoc(doc *html.Node) []Form {
 	if doc == nil {