Add "unset" value for PM prompt role, refactor icons display

227201a5fc981397792949033af8fc56ec1ce0ef

Cohee <18619528+Cohee1207@users.noreply.github.com>

Parents: d58f471cc379ad11e0a9861dd3d0a62e0939236d

public/css/promptmanager.css+4 -0 Viewed

@@ -28,6 +28,10 @@
28	28	color: var(--white50a);
29	29	}
30	30
	31	+#completion_prompt_manager #completion_prompt_manager_list .completion_prompt_manager_prompt .completion_prompt_manager_prompt_name .fa-solid[data-role] {
	32	+ vertical-align: unset;
	33	+}
	34	+
31	35	#completion_prompt_manager #completion_prompt_manager_list .completion_prompt_manager_prompt_invisible {
32	36	display: none;
33	37	}

public/index.html+1 -0 Viewed

                                 </label>
                                 <div class="text_muted" data-i18n="To whom this message will be attributed.">To whom this message will be attributed.</div>
                                 <select id="completion_prompt_manager_popup_entry_form_role" class="text_pole" name="role">
 +                                    <option data-i18n="Default (unset)" value="">Default (unset)</option>
                                     <option data-i18n="System" value="system">System</option>
                                     <option data-i18n="User" value="user">User</option>
                                     <option data-i18n="AI Assistant" value="assistant">AI Assistant</option>

public/scripts/PromptManager.js+12 -31 Viewed

         const forbidOverridesBlock = document.getElementById(this.configuration.prefix + 'prompt_manager_forbid_overrides_block');
         nameField.value = prompt.name ?? '';
         roleField.value = prompt.role ?? 'system';
         promptField.value = prompt.content ?? '';
         promptField.disabled = prompt.marker ?? false;
         injectionPositionField.value = prompt.injection_position ?? INJECTION_POSITION.RELATIVE;

+            }
             const encodedName = escapeHtml(prompt.name);
 +            const isRolePrompt = !!prompt.role;
             const isMarkerPrompt = prompt.marker && prompt.injection_position !== INJECTION_POSITION.ABSOLUTE;
             const isSystemPrompt = !prompt.marker && prompt.system_prompt && prompt.injection_position !== INJECTION_POSITION.ABSOLUTE && !prompt.forbid_overrides;
             const isImportantPrompt = !prompt.marker && prompt.system_prompt && prompt.injection_position !== INJECTION_POSITION.ABSOLUTE && prompt.forbid_overrides;

             const importantClass = isImportantPrompt ? `${prefix}prompt_manager_important` : '';
             //add role icons to the right of prompt name
+-
 +            const promptRoles = {
 -            const hasRole = !!prompt.role;
 +                system: { roleIcon: 'fa-cog', roleTitle: 'Prompt will be sent as System' },
 -            let roleIcon, roleTitle;
 +                assistant: { roleIcon: 'fa-robot', roleTitle: 'Prompt will be sent as Assistant' },
 -            if (hasRole) {
 +                user: { roleIcon: 'fa-user', roleTitle: 'Prompt will be sent as User' },
 -                switch (prompt.role) {
 +            };
 -                    case 'system':
 +            const roleIcon = isRolePrompt ? promptRoles[prompt.role]?.roleIcon : '';
 -                        roleIcon = 'fa-cog';
 +            const roleTitle = isRolePrompt ? promptRoles[prompt.role]?.roleTitle : '';
 -                        roleTitle = 'Prompt will be sent as System';
 -                        break;
 -                    case 'assistant':
 -                        roleIcon = 'fa-robot';
 -                        roleTitle = 'Prompt will be sent as Assistant';
 -                        break;
 -                    case 'user':
 -                        roleIcon = 'fa-user';
 -                        roleTitle = 'Prompt will be sent as User';
 -                }
+-
 -                // not sure if this makes sense to include as 'markers' should not be sendable except by System, included in case I'm wrong.
 -                // if it shouldn't be changeable, we should remove that dropdown from editor for Marker prompts.
 -                /*
 -                if (isMarkerPrompt && prompt.role !== 'user' && prompt.role !== 'assistant' && prompt.role !== 'system') {
 -                    roleIcon = 'fa-cog';
 -                    roleTitle = 'ST Global Prompt is sent as System by default (but can be changed in Editor)';
 -                }
 -                */
 -            }
             listItemHtml += `
                 <li class="${prefix}prompt_manager_prompt ${draggableClass} ${enabledClass} ${markerClass} ${importantClass}" data-pm-identifier="${escapeHtml(prompt.identifier)}">
                     <span class="${prefix}prompt_manager_prompt_name" data-pm-name="${encodedName}">
                         ${isMarkerPrompt ? '<span class="fa-fw fa-solid fa-thumb-tack" title="Marker"></span>' : ''}
                         ${isSystemPrompt ? '<span class="fa-fw fa-solid fa-square-poll-horizontal" title="Global Prompt"></span>' : ''}

                         ${isUserPrompt ? '<span class="fa-fw fa-solid fa-user" title="Preset Prompt"></span>' : ''}
                         ${isInjectionPrompt ? '<span class="fa-fw fa-solid fa-syringe" title="In-Chat Injection"></span>' : ''}
                         ${this.isPromptInspectionAllowed(prompt) ? `<a title="${encodedName}" class="prompt-manager-inspect-action">${encodedName}</a>` : `<span title="${encodedName}">${encodedName}</span>`}
                         ${hasRoleisRolePrompt ? `<span data-role="${escapeHtml(prompt.role)}" class='"fa-xs fa-solid ${roleIcon}'" title='"${roleTitle}'"></span>` : ''}
                         ${isInjectionPrompt ? `<small class="prompt-manager-injection-depth">@ ${escapeHtml(prompt.injection_depth)}</small>` : ''}
                         ${isOverriddenPrompt ? '<small class="fa-solid fa-address-card prompt-manager-overridden" title="Pulled from a character card"></small>' : ''}
                     </span>
                     <span>