private String serverAddress;
    private int flags;
    
    // Getters and setters...
}

class WitnessRegisterResponse {
    private String registrationId;
    private int returnCode;
    
    public boolean isSuccess() { return returnCode == 0; }
    public String getError() { return "Error code: " + returnCode; }
    // Getters and setters...
}

class WitnessUnregisterRequest {

        return someProperty
    }
}
```

It is not acceptable to introduce new properties on a task, extension or domain object that use plain getters and setters.  It is also not acceptable to add setters that take a Provider.

as an intermediate step.

#### Vars and Vals

Java doesn’t have language support for properties so developers make do with getters and setters.
Kotlin does have properties and we take advantage of them in OkHttp.

 * **Address**: certificatePinner, connectionSpecs, dns, hostnameVerifier, protocols, proxy,

        // This method should always return 0 as the request doesn't read responses
        assertEquals(0, bytesRead);
    }

    @Test
    @DisplayName("Test complete flow with all setters")
    void testCompleteFlow() {
        request = new Smb2SetInfoRequest(mockConfig);

        // Set all fields
        byte[] newFileId = new byte[16];
        for (int i = 0; i < 16; i++) {

        assertTrue("Single non-hiragana char query should match first", result);
    }

    // ============================================================
    // Tests for configuration setters
    // ============================================================

    @Test
    public void test_setSkipDuplicateWords() {
        request.setSkipDuplicateWords(true);
        // Verify setting doesn't throw

        assertEquals(Integer.MIN_VALUE, searchRenderData.getCurrentPageNumber());
    }

    @Test
    public void test_nullSafety() {
        // Ensure all getters return appropriate values when setters haven't been called
        SearchRenderData freshData = new SearchRenderData();

        // These should return null
        assertNull(freshData.getDocumentItems());

    private WordManagementOperations wordMgmtOps;
    private ContentIndexingOperations contentOps;

    /** Flag indicating that operations need to be re-initialized. Not thread-safe: setters are configuration-time only. */
    private volatile boolean operationsStale = false;

    /**
     * Constructor for SuggestIndexer.
     * @param client The OpenSearch client.
     * @param index The index name.

        assertNotNull(response);
        // Should handle special characters gracefully
    }

    // ============================================================
    // Tests for setters with null values
    // ============================================================

    @Test
    public void test_setNullIndexName() throws Exception {
        SuggestIndexer indexer = suggester.indexer();

        // This method should always return 0 as the request doesn't read responses
        assertEquals(0, bytesRead);
    }

    @Test
    @DisplayName("Test complete flow with all setters")
    void testCompleteFlow() {
        request = new Smb2QueryInfoRequest(mockConfig);

        // Set all fields
        byte[] newFileId = new byte[16];
        for (int i = 0; i < 16; i++) {

   2. Add Javadoc to existing code if you can.
   3. Document the "why", not the "how", unless that's important to the
      "why".
   4. Don't document anything trivial or obvious (e.g. getters and
      setters). In other words, the Javadoc should add some value.

#### The long version

   1. If you add a new Java package, please also add package-level