fix: correct knowledge block canonical pair pattern and subblock migration

- Rename manualDocumentId to documentId (advanced subblock ID should match
  canonicalParamId, consistent with airtable/gmail patterns)
- Fix documentSelector.dependsOn to reference knowledgeBaseSelector (basic
  depends on basic, not advanced)
- Remove unnecessary documentId migration (ID unchanged from main)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

apps/sim/blocks/blocks/knowledge.ts

@@ -127,7 +127,7 @@ export const KnowledgeBlock: BlockConfig = {
       type: 'document-selector',
       canonicalParamId: 'documentId',
       placeholder: 'Select document',
-      dependsOn: ['knowledgeBaseId'],
+      dependsOn: ['knowledgeBaseSelector'],
       required: true,
       mode: 'basic',
       condition: {
@@ -144,7 +144,7 @@ export const KnowledgeBlock: BlockConfig = {
     },
     // Document selector — advanced mode (manual ID input)
     {
-      id: 'manualDocumentId',
+      id: 'documentId',
       title: 'Document ID',
       type: 'short-input',
       canonicalParamId: 'documentId',