Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
Expand Up@@ -30,6 +30,7 @@ export class EVaultService {
return freshData;
}


/**
* Get cache status information
*/
Expand Down
2 changes: 1 addition & 1 deletion infrastructure/control-panel/src/lib/services/registry.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,7 +19,7 @@ export class RegistryService {
private baseUrl: string;

constructor() {
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.staging.metastate.foundation';
this.baseUrl = env.PUBLIC_REGISTRY_URL || 'https://registry.w3ds.metastate.foundation';
}

async getEVaults(): Promise<RegistryVault[]> {
Expand Down
6 changes: 5 additions & 1 deletion infrastructure/control-panel/src/routes/+page.svelte
Original file line numberDiff line numberDiff line change
Expand Up@@ -278,7 +278,11 @@
isLoading = true;
error = null;
console.log('🔍 Fetching eVaults...');
const data = await EVaultService.getEVaults();
let data = await EVaultService.forceRefresh();
if (!Array.isArray(data) || data.length === 0) {
// Fallback to cache only if fresh load returns empty/invalid.
data = await EVaultService.getEVaults();
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
console.log('📦 Data received:', data);
console.log('📊 Data type:', typeof data);
console.log('📊 Is array:', Array.isArray(data));
Expand Down
76 changes: 52 additions & 24 deletions infrastructure/control-panel/src/routes/api/evaults/+server.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -41,16 +41,20 @@ function firstNonEmptyString(...values: unknown[]): string | null {
async function fetchFirstParsedByOntology(
vault: RegistryVault,
ontologyId: string,
token: string
token?: string
): Promise<Record<string, unknown> | null> {
try {
const tryRequest = async (withAuth: boolean): Promise<Record<string, unknown> | null> => {
const headers: Record<string, string> = {
'Content-Type': 'application/json',
'X-ENAME': vault.ename
};
if (withAuth && token) {
headers.Authorization = `Bearer ${token}`;
}

const response = await fetch(`${vault.uri}/graphql`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-ENAME': vault.ename,
Authorization: `Bearer ${token}`
},
headers,
body: JSON.stringify({
query: META_ENVELOPES_QUERY,
variables: {
Expand All@@ -66,21 +70,38 @@ async function fetchFirstParsedByOntology(
}

const payload = await response.json();
if (Array.isArray(payload?.errors) && payload.errors.length > 0) {
return null;
}
const parsed = payload?.data?.metaEnvelopes?.edges?.[0]?.node?.parsed;

if (!parsed || typeof parsed !== 'object') {
return null;
}

return parsed as Record<string, unknown>;
};

try {
// Try with token first (preferred), then fallback to X-ENAME-only.
const withAuth = await tryRequest(Boolean(token));
if (withAuth) {
return withAuth;
}

if (token) {
return await tryRequest(false);
}

return null;
} catch {
return null;
}
}

async function resolveVaultIdentity(
vault: RegistryVault,
token: string
token?: string
): Promise<{ name: string; type: 'user' | 'group' }> {
const defaultName = firstNonEmptyString(vault.ename, vault.evault, 'Unknown') || 'Unknown';

Expand All@@ -99,7 +120,15 @@ async function resolveVaultIdentity(
return {
type: 'group',
name:
firstNonEmptyString(groupManifest.name, groupManifest.eName, vault.ename, vault.evault) ||
firstNonEmptyString(
groupManifest.name,
groupManifest.displayName,
groupManifest.title,
groupManifest.eName,
groupManifest.ename,
vault.ename,
vault.evault
) ||
defaultName
};
}
Expand DownExpand Up@@ -136,7 +165,12 @@ async function requestPlatformToken(platform: string): Promise<string> {
export const GET: RequestHandler = async ({ url }) => {
try {
const platform = env.PUBLIC_CONTROL_PANEL_URL || url.origin;
const token = await requestPlatformToken(platform);
let token: string | undefined;
try {
token = await requestPlatformToken(platform);
} catch (tokenError) {
console.warn('Falling back to X-ENAME-only eVault queries:', tokenError);
}

// Fetch all evaults from registry
const registryVaults = await registryService.getEVaults();
Expand All@@ -149,32 +183,26 @@ export const GET: RequestHandler = async ({ url }) => {

const identity = await resolveVaultIdentity(vault, token);

// Check health status by attempting to fetch from URI
let status = 'Unknown';
try {
const healthResponse = await fetch(`${vault.uri}/health`, {
signal: AbortSignal.timeout(2000) // 2 second timeout
});
status = healthResponse.ok ? 'Active' : 'Inactive';
} catch {
status = 'Inactive';
}

return {
id: evaultId,
name: identity.name,
type: identity.type,
ename: vault.ename,
uri: vault.uri,
evault: vault.evault,
status: status,
status: 'Unknown',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash# Check if health check logic was previously implemented and intentionally removed
git log --oneline -n 20 --all -- "infrastructure/control-panel/src/routes/api/evaults/+server.ts"2>/dev/null | head -10
# Search for any remaining health check references in the codebase
rg -n "health" --type=ts -g '!node_modules' infrastructure/control-panel/

Repository: MetaState-Prototype-Project/prototype

Length of output: 1644


🏁 Script executed:

# View the actual file around line 193 to see context
cat -n infrastructure/control-panel/src/routes/api/evaults/+server.ts | sed -n '180,200p'

Repository: MetaState-Prototype-Project/prototype

Length of output: 761


🏁 Script executed:

# Check the specific commit that changed this file
git show ad49195:infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -50

Repository: MetaState-Prototype-Project/prototype

Length of output: 1599


🏁 Script executed:

# Check if EVaultList.svelte still exists and uses the status field
find infrastructure/control-panel -name "EVaultList.svelte" -o -name "*EVaultList*"

Repository: MetaState-Prototype-Project/prototype

Length of output: 146


🏁 Script executed:

# Check if EVaultList.svelte uses the status field
cat -n infrastructure/control-panel/src/lib/components/EVaultList.svelte | grep -A 5 -B 5 "status"

Repository: MetaState-Prototype-Project/prototype

Length of output: 863


🏁 Script executed:

# Get the diff of the commit that made this change
git show ad49195 -- infrastructure/control-panel/src/routes/api/evaults/+server.ts | head -100

Repository: MetaState-Prototype-Project/prototype

Length of output: 2981


Status is hardcoded, losing health visibility in the eVaults list.

The hardcoded status: 'Unknown' means all eVaults display gray badges in EVaultList.svelte (lines 185-192) regardless of actual health. The component uses status for color-coded indicators ('Active' → green, 'Inactive' → red, 'Unknown' → gray).

While health check logic exists in the related details/+server.ts endpoint (fetching from {evault.uri}/health), it was not implemented in this list endpoint. The type comment "derived from health check" should be clarified since status is currently hardcoded rather than computed.

Consider either:

  1. Implementing actual health checks (following the pattern in details/+server.ts)
  2. Updating the type comment to clarify status is a placeholder, with a follow-up issue to implement health monitoring
  3. If health checks were intentionally omitted due to performance/caching concerns, document the rationale
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@infrastructure/control-panel/src/routes/api/evaults/`+server.ts at line 193,
The eVault list endpoint is hardcoding status: 'Unknown' (losing real health
visibility); update the +server.ts handler that builds the eVault objects to
compute status from the same health-check logic used in details/+server.ts (call
{evault.uri}/health or reuse a helper if one exists), map health responses to
'Active'/'Inactive'/'Unknown' and set that value instead of the literal, or
alternatively change the type comment to note that status is a placeholder and
add a TODO to implement health checks or document why checks are omitted
(reference EVaultList.svelte which relies on status and details/+server.ts where
health is implemented to guide the change).

serviceUrl: vault.uri
};
})
);

console.log(`Total eVaults found: ${evaults.length}`);
return json({ evaults });
// Filter out platform-owned user profiles (e.g. "File Manager Platform").
const filteredEVaults = evaults.filter(
(vault) => !(vault.type === 'user' && /platform$/i.test(vault.name.trim()))
);

console.log(`Total eVaults found: ${filteredEVaults.length}`);
return json({ evaults: filteredEVaults });
} catch (error) {
console.error('Error fetching eVaults:', error);
return json({ error: 'Failed to fetch eVaults', evaults: [] }, { status: 500 });
Expand Down