Function cbm_upsert_json_named_mcp

src/cli/cli.c:2062–2164

}

static int cbm_upsert_json_named_mcp(const char *binary_path, const char *config_path,
                                     const char *const *object_path, size_t path_len,
                                     cbm_json_mcp_schema_t schema, const char *entry_name,
                                     const char *argument) {
    if (!binary_path || !config_path || !object_path || !entry_name || !entry_name[0]) {
        return CLI_ERR;
    }
    char *document = NULL;
    size_t document_length = 0U;
    int read_result = cbm_json_like_read_document(config_path, &document, &document_length);
    if (read_result < 0) {
        return CLI_ERR;
    }
    if (read_result == 0) {
        char *command = NULL;
        int ownership = cbm_json_mcp_snapshot_ownership(
            document, document_length, object_path, path_len, schema, entry_name, argument,
            binary_path, g_previous_managed_mcp_command, &command);
        /* An entry that already says what we would say, but carries extra keys
         * the client added, is ALREADY SATISFIED. Return success without
         * touching the file.
         *
         * We must not rewrite it: the editor replaces an entry wholesale, so
         * writing our canonical shape over it would delete those keys. Doing
         * nothing is both correct and lossless — the entry already points at
         * this binary with the right type, which is the whole content of the
         * install.
         *
         * This is #1630: OpenCode writes `"enabled": true` next to our
         * `command` and `type`, so every user who had toggled a server in the
         * UI hit `op=mcp_install` failure. Confirmed on Linux and Windows with
         * two independent configs. Merging our fields into an annotated entry
         * while preserving the rest is the fuller fix and is tracked there;
         * this makes the common case work without risking anyone's config. */
        if (ownership == CBM_JSON_LIKE_OBJECT_MATCH_WITH_EXTRAS) {
            /* Owned via the PREVIOUS managed binary during a relocating
             * update: the annotated entry still names the old location, and a
             * wholesale rewrite would drop the client's keys — repair only the
             * command member (#1630's deferred field-merge). An entry already
             * naming the current binary needs nothing. */
            bool via_previous = command && g_previous_managed_mcp_command &&
                                strcmp(command, g_previous_managed_mcp_command) == 0 &&
                                strcmp(command, binary_path) != 0;
            if (!via_previous) {
                free(command);
                free(document);
                return CLI_OK;
            }
            char *value = cbm_json_mcp_render_command_value(binary_path, schema);
            if (!value) {
                free(command);
                free(document);
                return CLI_ERR;
            }
            int repair = cbm_json_like_replace_field_raw_if_unchanged(
                config_path, object_path, path_len, entry_name, "command", value, document,
                document_length);
            free(value);