assertEquals(ChatIntent.UNCLEAR, ChatIntent.fromValue("unclear"));
        assertEquals(ChatIntent.UNCLEAR, ChatIntent.fromValue("UNCLEAR"));
        assertEquals(ChatIntent.UNCLEAR, ChatIntent.fromValue("Unclear"));
    }

    @Test
    public void test_fromValue_null() {
        // Defaults to UNCLEAR when value is null
        assertEquals(ChatIntent.UNCLEAR, ChatIntent.fromValue(null));
    }

    @Test

    /** User wants a summary of a specific document */
    SUMMARY("summary"),

    /** User is asking a FAQ-type question */
    FAQ("faq"),

    /** Intent is unclear - need to ask user for clarification */
    UNCLEAR("unclear");

    private final String value;

    ChatIntent(final String value) {
        this.value = value;
    }

    /**
     * Returns the string value of this intent.
     *

    }

    /**
     * Creates an unclear intent result when intent cannot be determined.
     *
     * @param reasoning the detection reasoning
     * @return the unclear intent result
     */
    public static IntentDetectionResult unclear(final String reasoning) {
        return new IntentDetectionResult(ChatIntent.UNCLEAR, null, null, reasoning);
    }

    /**

    @Test
    public void test_unclear() {
        final IntentDetectionResult result = IntentDetectionResult.unclear("unclear intent");

        assertEquals(ChatIntent.UNCLEAR, result.getIntent());
        assertNull(result.getQuery());
        assertNull(result.getDocumentUrl());
        assertEquals("unclear intent", result.getReasoning());
    }

    @Test
    public void test_unclear_withNullReasoning() {

## Reviewing PRs

Reviewers ensure MinIO’s commit history remains a clear, reliable record. Responsibilities include:

1. **Commit Quality**:
   - Verify each commit explains *why* the change was made (e.g., “So that…”).
   - Request rebasing if commits are unclear, redundant, or lack context (e.g., “Please squash typo fixes into the parent commit”).

2. **Code Quality**:

   * with {@code FeatureSpecificTestSuiteBuilder.suppressing()} until <a
   * href="https://bugs.openjdk.org/browse/JDK-6409434">JDK-6409434</a> is fixed. It's unclear
   * whether nulls were to be permitted or forbidden, but presumably the eventual fix will be to
   * permit them, as it seems more likely that code would depend on that behavior than on the other.

   * with {@code FeatureSpecificTestSuiteBuilder.suppressing()} until <a
   * href="https://bugs.openjdk.org/browse/JDK-6409434">JDK-6409434</a> is fixed. It's unclear
   * whether nulls were to be permitted or forbidden, but presumably the eventual fix will be to
   * permit them, as it seems more likely that code would depend on that behavior than on the other.

   * have started execution. This applies also to tasks given to {@code invokeAll} or {@code
   * invokeAny} which are pending serial execution, even the subset of the tasks that have not yet
   * started execution. It is unclear from the {@code ExecutorService} specification if these should
   * be included, and it's much easier to implement the interpretation that they not be. Finally, a

   * with {@code FeatureSpecificTestSuiteBuilder.suppressing()} until <a
   * href="https://bugs.openjdk.org/browse/JDK-6409434">JDK-6409434</a> is fixed. It's unclear
   * whether nulls were to be permitted or forbidden, but presumably the eventual fix will be to
   * permit them, as it seems more likely that code would depend on that behavior than on the other.

   * with {@code FeatureSpecificTestSuiteBuilder.suppressing()} until <a
   * href="https://bugs.openjdk.org/browse/JDK-6409434">JDK-6409434</a> is fixed. It's unclear
   * whether nulls were to be permitted or forbidden, but presumably the eventual fix will be to
   * permit them, as it seems more likely that code would depend on that behavior than on the other.