samymsa

Hi Maxwell,

Here are some issues I flagged if it can help.

Issue 1 — Orphaned chunks on unpublish

When a published post is changed to draft/pending/private, the unpublish path at line 5152 calls mxchat_delete_from_pinecone_by_url() which only deletes the base vector (md5(url)). If the content was stored chunked, all {md5(url)}_chunk_0, _chunk_1, etc. vectors remain in Pinecone and will continue polluting search results.

Compare to the delete/trash path at line 5415 which correctly calls the chunk-aware MxChat_Utils::delete_chunks_for_url().

Issue 2 — Same problem for WooCommerce products

mxchat_handle_product_delete at line 5629 also uses the non-chunk-aware mxchat_delete_from_pinecone_by_url(). Trashing or deleting a product with chunked content leaves orphaned chunk vectors.

Issue 3 — Trashed permalink mismatch

When wp_trash_post fires, WordPress may have already appended __trashed to the post slug. So get_permalink($post_id) at line 5408 returns something like:

https://example.com/my-post__trashed/

Since the vector ID is md5($source_url), this won’t match the original vector ID stored in Pinecone. The delete request targets a non-existent ID and the real entries silently survive.

This affects both mxchat_handle_post_delete and mxchat_handle_product_delete.

Hi,

Thanks for your answer.

To clarify, I think that if a content is in the knowledge base, the knowledge information associated to it should be updated automatically. This is because if I add some content in the knowledge base, then update the content with auto sync turned off, the knowledge information will be outdated.

I understand my ideal workflow is not the same as for other users of MxChat, but a simple hook might do the trick. I’ll send you a code snippet when I have some time.

However, I had auto sync turned on for “Pages”, and the pages knowledge information did not get updated. I am using Pinecone. Something might be wrong in the current implementation.

Thanks for all the work,

Samy

You’re welcome, happy to help MxChat become better!

Thank your for the fixes!

Closing this topic and opening a dedicated one about dynamic data / shortcodes inside system prompt.

Thanks again for you support @m4xw3ll !

Hi Maxwell,

Following up our conversation, would it be possible for you to add a hook that allows to alter the system_prompt_instructions, or maybe simpler to allow shortcodes inside the system prompt (not into the current message context)?

This way we would be able to add dynamic data into the system prompt. I’m currently forced to use a mu-plugin and hook into the get_options, which is not ideal.

Best,
Samy

I’ll try this, thanks for your quick answer!

Thanks so much!

For your information, here is the code I added:

In your plugin, inside mxchat-basic/admin/class-knowledge-manager.php line 2792:

// Apply filter to allow modification of post data before processing
// This allows me and other developpers to alter the behaviour without modifying mxchat code
$post = apply_filters('mxchat_process_post_data', $post);

In my custom theme:

/**
 * Enrich post_content with whitelisted post meta for MXChat embeddings
 */

function mxchat_enrich_post_content_with_meta($post)

{

  $meta_map = [
    // EVENT META KEYS
    'event_start_date' => 'Start Date',
    'event_start_time' => 'Start Time',
    'event_end_date' => 'End Date',
    'event_end_time' => 'End Time',

  ];

  // Return early if not a WP_Post object
  if (!($post instanceof WP_Post)) {
    return $post;
  }

  $lines = [];

  foreach ($meta_map as $meta_key => $label) {

    $value = get_post_meta($post->ID, $meta_key, true);

    if (empty($value)) {
      continue;
    }

    // Normalize value

    if (is_array($value)) {
      $value = implode(', ', array_map('wp_strip_all_tags', $value));
    } else {
      $value = wp_strip_all_tags((string) $value);
    }

    if ($value !== '') {
      $lines[] = $label . ': ' . $value;
    }
  }

  if (!empty($lines)) {
    $post->post_content .= "\n\n" . implode("\n", $lines);
  }

  return $post;

}

add_filter('mxchat_process_post_data', 'mxchat_enrich_post_content_with_meta');

Feel free to take the best approach 😉

• This reply was modified 4 months, 1 week ago by samymsa.