fix: The detected filetype is PLAIN_TEXT, but the provided filetype was HTML

[github-actions]

Closes #6873

Summary

Issue #6873 Fix Summary

Problem

When using Goose with the AWS Bedrock provider, viewing .html files containing template syntax (Jinja2, Django, etc.) caused a ValidationException error:
"The detected filetype is PLAIN_TEXT, but the provided filetype was HTML."

This occurred because the to_bedrock_document() function determined the document format based on file extension. When it encountered .html files, it declared them as Html format to Bedrock's API. However, Bedrock validates content against the declared format, and template files containing {{ }} or {% %} syntax aren't valid HTML according to Bedrock's content validator.

Solution

Removed HTML from the list of supported document formats in the to_bedrock_document() function. HTML files now fall through to the default case (_ => return Ok(None)), which means they are handled as plain text content instead of being declared as HTML documents.

Change Made

File: crates/goose/src/providers/formats/bedrock.rs

Removed line:

Some((name, "html")) => (name, bedrock::DocumentFormat::Html),

From the match statement at line 258-263:

// Before:
let (name, format) = match filename.split_once('.') {
    Some((name, "txt")) => (name, bedrock::DocumentFormat::Txt),
    Some((name, "csv")) => (name, bedrock::DocumentFormat::Csv),
    Some((name, "md")) => (name, bedrock::DocumentFormat::Md),
    Some((name, "html")) => (name, bedrock::DocumentFormat::Html),  // REMOVED
    _ => return Ok(None), // Not a supported document type
};

// After:
let (name, format) = match filename.split_once('.') {
    Some((name, "txt")) => (name, bedrock::DocumentFormat::Txt),
    Some((name, "csv")) => (name, bedrock::DocumentFormat::Csv),
    Some((name, "md")) => (name, bedrock::DocumentFormat::Md),
    _ => return Ok(None), // Not a supported document type
};

Verification

• ✅ cargo check - passed
• ✅ cargo fmt - passed
• ✅ ./scripts/clippy-lint.sh - passed (pre-existing warnings only)

Impact

• HTML files will now be handled as plain text content, avoiding Bedrock's content validation
• No impact on other file types (txt, csv, md)
• This is the minimal safe fix as suggested in the issue comments

---

Generated by goose Issue Solver

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[katzdave]

Seems fine to delete since scoped to bedrock.