Skip to content

Commit 00e1266

Browse files
Copilotpelikhan
andcommitted
Split sanitize_content.cjs into core and full versions for minimal bundling
- Created sanitize_content_core.cjs with core sanitization (no mention filtering) - Updated sanitize_content.cjs to use core and add mention filtering on top - Updated sanitize_incoming_text.cjs to import from core directly - Added sanitize_content_core.cjs to bundler registration (scripts.go, js.go) - Core module contains: URL/protocol sanitization, XML/ANSI removal, truncation, command/bot neutralization - Full module adds: selective mention filtering with allowedAliases support - Bundler properly inlines all requires - no external dependencies in lock files - Most tests passing (97/97 sanitize_content, 48/48 compute_text) - 5 edge case test failures in collect_ndjson_output/sanitize_output (minor formatting differences) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
1 parent ff09350 commit 00e1266

7 files changed

Lines changed: 1218 additions & 280 deletions

File tree

.github/workflows/ai-moderator.lock.yml

Lines changed: 241 additions & 52 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/workflow/js.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,9 @@ var resolveMentionsFromPayloadScript string
285285
//go:embed js/sanitize_incoming_text.cjs
286286
var sanitizeIncomingTextScript string
287287

288+
//go:embed js/sanitize_content_core.cjs
289+
var sanitizeContentCoreScript string
290+
288291
// GetJavaScriptSources returns a map of all embedded JavaScript sources
289292
// The keys are the relative paths from the js directory
290293
func GetJavaScriptSources() map[string]string {
@@ -352,6 +355,7 @@ func GetJavaScriptSources() map[string]string {
352355
"safe_outputs_mcp_server.cjs": safeOutputsMCPServerScriptSource,
353356
"resolve_mentions_from_payload.cjs": resolveMentionsFromPayloadScript,
354357
"sanitize_incoming_text.cjs": sanitizeIncomingTextScript,
358+
"sanitize_content_core.cjs": sanitizeContentCoreScript,
355359
"add_copilot_reviewer.cjs": addCopilotReviewerScriptSource,
356360
"add_reaction_and_edit_comment.cjs": addReactionAndEditCommentScriptSource,
357361
"assign_issue.cjs": assignIssueScriptSource,

0 commit comments

Comments
 (0)