← All changes | classes/core.php +2 -161 0.5.3 → 0.4.5 View file →

		@@ -14,12 +14,8 @@
14	14	public $site_url = null;
15	15	public $mwcode = null;
16	16	public $licenser = null;
17	17
18		- // IDs of global snippets already executed this request (by the plugins_loaded pass
19		- // or by load_global_snippets), so a global never runs twice and never re-declares.
20		- public $loaded_global_ids = [];
21		-
22	18	private $option_name = 'mwcode_options';
23	19
24	20	public function __construct() {
25	21	global $mwcode;
		@@ -106,9 +102,8 @@
106	102
107	103	//AI
108	104	"ai_suggestions" => false,
109	105	"ai_engine_status"=> false,
110		- "mwai_active" => false,
111	106	"ai_engine_message" => "",
112	107
113	108	//API
114	109	"api_endpoint" => false,
		@@ -115,9 +110,8 @@
115	110	"api_token" => md5( time() . rand() ),
116	111
117	112	//MCP
118	113	"mcp_support" => false,
119		- "mcp_functions" => false,
120	114
121	115	//MAINTENANCE
122	116	"clean_uninstall" => false,
123	117	];
		@@ -138,9 +132,9 @@
138	132
139	133	$options = $this->sanitize_options( $options );
140	134
141	135	if ( !update_option( $this->option_name, $options, false ) ) {
142		- //$this->log( '💾 There was an issue updating the options.' );
	136	+ $this->log( '💾 There was an issue updating the options.' );
143	137	}
144	138
145	139	return $options;
146	140	}
		@@ -191,12 +185,8 @@
191	185
192	186	private function updateAIEngineStatus( &$options ) {
193	187	global $mwai;
194	188
195		- // AI Engine is active (regardless of whether an API key is configured).
196		- // MCP exposure only needs AI Engine present, not a key, so the MCP toggles
197		- // gate on this rather than on mwai_has_ai.
198		- $options['mwai_active'] = !empty( $mwai );
199	189	$options['mwai_has_ai'] = !empty( $mwai ) && method_exists( $mwai, 'hasAI' ) && $mwai->hasAI();
200	190	// Legacy
201	191	$options['ai_engine_status'] = $options['mwai_has_ai'];
202	192
		@@ -230,24 +220,9 @@
230	220
231	221	$this->snippet->validate( $params );
232	222
233	223	$params = $this->snippet->formatParamsForDatabase( $params );
234		-
235		- // Route to UPDATE when an existing snippet id is provided (updateSnippet / the
236		- // MCP mwcode_update_snippet tool). This previously always insert()ed, so an
237		- // update tried to INSERT a row with an already-used primary key: that fails on
238		- // the SQLite backend (Studio/Playground) with "Could not insert the snippet",
239		- // and duplicates or errors elsewhere. The admin UI was unaffected because it
240		- // calls snippet->update() directly.
241		- $existing = !empty( $params['id'] ) ? $this->snippet->select_one( $params['id'] ) : null;
242		- if ( $existing ) {
243		- $this->snippet->update( $params );
244		- $result = $params['id'];
245		- }
246		- else {
247		- unset( $params['id'] );
248		- $result = $this->snippet->insert( $params );
249		- }
	224	+ $result = $this->snippet->insert( $params );
250	225	$snippet = $this->snippet->select_one( $result );
251	226
252	227	if( $result ) {
253	228	$params['id'] = (string)$result;
		@@ -407,18 +382,8 @@
407	382
408	383	$this->log( '⚡ Arguments provided: ' . json_encode( $args ) );
409	384	}
410	385
411		- // Global snippets are meant to be always accessible. On non-whitelisted REST routes
412		- // (Workflow Engine, MCP, AI function-calling) the plugins_loaded pass blocks them, so
413		- // make sure their helper library is loaded before we run a function that may call it.
414		- $this->load_global_snippets();
415		-
416		- // Make every *other* active PHP function snippet available so this function can
417		- // call its siblings. We pass the current name as the exception so the target is
418		- // still defined below (with the edited/test code when testing), not pre-defined here.
419		- $this->define_all_functions( $params['name'] );
420		-
421	386	// Check if the function has already been defined
422	387	if ( !in_array( $params['name'], $defined_functions ) ) {
423	388
424	389	// If not, proceed with modification and definition
		@@ -530,132 +495,8 @@
530	495	return $lint;
531	496	}
532	497
533	498	return null;
534		- }
535		-
536		- /**
537		- * Load the active global snippets (persistent + backend/frontend for this context)
538		- * that haven't already run this request, so on-demand function execution has the same
539		- * always-available helper library a normal page load would. Callable functions are
540		- * typically small wrappers around these globals.
541		- *
542		- * On non-whitelisted REST routes (Workflow Engine, MCP, AI function-calling) the
543		- * plugins_loaded pass blocks global snippets for safety; this restores them for the
544		- * deliberate, authorized act of executing a snippet. The loaded-id registry guarantees
545		- * each global runs at most once per request, so nothing is ever re-declared.
546		- */
547		- function load_global_snippets() {
548		- global $current_mwcode_snippet;
549		- static $done = false;
550		- if ( $done ) {
551		- return;
552		- }
553		- $done = true;
554		-
555		- if ( empty( $this->snippet ) ) {
556		- $this->snippet = new Meow_MWCODE_Modules_Snippet( $this );
557		- }
558		-
559		- $scope = is_admin() ? [ 'backend', 'persistent' ] : [ 'frontend', 'persistent' ];
560		-
561		- $snippets = $this->snippet->select(
562		- null, // offset
563		- -1, // limit (all)
564		- [
565		- [ 'accessor' => 'active', 'value' => 1 ],
566		- [ 'accessor' => 'scope', 'value' => $scope ],
567		- ],
568		- [ 'accessor' => 'priority', 'by' => 'DESC' ]
569		- )['data'] ?? [];
570		-
571		- foreach ( $snippets as $snippet ) {
572		- // Skip globals already executed this request (e.g. by the plugins_loaded pass).
573		- if ( in_array( $snippet['id'], $this->loaded_global_ids ) ) {
574		- continue;
575		- }
576		- $this->loaded_global_ids[] = $snippet['id'];
577		-
578		- $code = $this->snippet->sanitize_code( $snippet['code'] );
579		- $current_mwcode_snippet = $snippet;
580		- try {
581		- ob_start();
582		- eval( $code );
583		- ob_end_clean();
584		- } catch ( Throwable $e ) {
585		- ob_end_clean();
586		- $this->log( "⚠️ Code Engine: Failed to load global snippet \"{$snippet['name']}\": " . $e->getMessage() );
587		- }
588		- }
589		- $current_mwcode_snippet = null;
590		- }
591		-
592		- /**
593		- * Declare every active PHP function snippet in the current request, without
594		- * invoking any of them, so function snippets can call one another.
595		- *
596		- * Function snippets are not auto-loaded on every request (unlike global/backend/
597		- * frontend scopes) — they are meant to run on demand. This is the PHP counterpart
598		- * to get_js_functions_to_push(): it makes the whole library of functions callable
599		- * before a function is executed (via REST, MCP, AI function-calling, Workflow Engine).
600		- *
601		- * Idempotent: a static guard runs the full pass only once per request, and each
602		- * definition is wrapped in function_exists() so nothing is ever redefined.
603		- *
604		- * @param string|null $except Function name to skip (the one run_snippet is about to
605		- * define itself, so edited/test code keeps priority).
606		- */
607		- function define_all_functions( $except = null ) {
608		- static $loaded = false;
609		- if ( $loaded ) {
610		- return;
611		- }
612		- $loaded = true;
613		-
614		- if ( empty( $this->snippet ) ) {
615		- $this->snippet = new Meow_MWCODE_Modules_Snippet( $this );
616		- }
617		-
618		- // One query for every active function snippet (code included), then enrich with
619		- // the function metadata (name + target) the same way run_snippet does.
620		- $snippets = $this->snippet->select(
621		- null, // offset
622		- -1, // limit (all)
623		- [
624		- [ 'accessor' => 'active', 'value' => 1 ],
625		- [ 'accessor' => 'scope', 'value' => 'function' ],
626		- ],
627		- [] // sort
628		- )['data'] ?? [];
629		-
630		- if ( empty( $snippets ) ) {
631		- return;
632		- }
633		-
634		- $this->snippet->get_function_snippets_data( $snippets );
635		-
636		- foreach ( $snippets as $snippet ) {
637		- $name = $snippet['functionName'] ?? '';
638		- $target = strtolower( $snippet['functionTarget'] ?? 'php' );
639		-
640		- // Skip JS functions (pushed to the front-end separately), the function the
641		- // caller will define itself, and anything already declared in this request.
642		- if ( $name === '' || $target === 'js' || $name === $except || function_exists( $name ) ) {
643		- continue;
644		- }
645		-
646		- // Mirror run_snippet()'s non-test handling: drop echo statements, then declare
647		- // (never call) the function, guarded so a later run_snippet() call is a no-op.
648		- $code = $this->snippet->sanitize_code( $snippet['code'] );
649		- $code = preg_replace( '/echo\s+(.+?);/s', '', $code );
650		- $code = "if (!function_exists('{$name}')) {\n{$code}\n}\n";
651		-
652		- try {
653		- eval( $code );
654		- } catch ( Throwable $e ) {
655		- $this->log( "⚠️ Code Engine: Failed to pre-define function \"{$name}\": " . $e->getMessage() );
656		- }
657		- }
658	499	}
659	500
660	501	public function get_js_functions_to_push() {
661	502	$functions = $this->snippet->get_functions();