Skip to content

Commit 34837da

Browse files
committed
fix: resolve lint errors and missing dependencies on mcp-deploy-tools
1 parent 00e20c4 commit 34837da

3 files changed

Lines changed: 19 additions & 7 deletions

File tree

src/mcp/apps/update_environment/mcp-app.ts

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
1-
import { App, applyDocumentTheme, applyHostStyleVariables, applyHostFonts } from "@modelcontextprotocol/ext-apps";
1+
import {
2+
App,
3+
applyDocumentTheme,
4+
applyHostStyleVariables,
5+
applyHostFonts,
6+
} from "@modelcontextprotocol/ext-apps";
27

38
const app = new App({ name: "Update Firebase Environment", version: "1.0.0" });
49

@@ -13,7 +18,6 @@ let selectedProjectId: string | null = null;
1318

1419
const envProjectIdEl = document.getElementById("env-project-id") as HTMLSpanElement;
1520
const envUserEl = document.getElementById("env-user") as HTMLSpanElement;
16-
const currentEnvBox = document.getElementById("current-env") as HTMLDivElement;
1721

1822
function showStatus(message: string, type: "success" | "error" | "info") {
1923
statusBox.textContent = message;
@@ -99,7 +103,8 @@ submitBtn.onclick = async () => {
99103
}
100104
};
101105

102-
app.ontoolresult = (result) => {
106+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
107+
app.ontoolresult = (_result) => {
103108
// We can handle tool results if needed, but we rely on manual triggers for list_projects
104109
};
105110

@@ -120,7 +125,10 @@ app.onhostcontextchanged = (ctx) => {
120125

121126
// Fetch current environment
122127
try {
123-
const envResult = await app.callServerTool({ name: "firebase_get_environment", arguments: {} });
128+
const envResult = await app.callServerTool({
129+
name: "firebase_get_environment",
130+
arguments: {},
131+
});
124132
const envData = envResult.structuredContent as any;
125133
if (envData) {
126134
envProjectIdEl.textContent = envData.projectId || "<NONE>";
@@ -140,7 +148,9 @@ app.onhostcontextchanged = (ctx) => {
140148
filteredProjects = projects;
141149
renderProjects();
142150
showStatus("Projects loaded successfully.", "success");
143-
setTimeout(() => { if (statusBox.className === "status success") statusBox.style.display = "none"; }, 3000);
151+
setTimeout(() => {
152+
if (statusBox.className === "status success") statusBox.style.display = "none";
153+
}, 3000);
144154
} else {
145155
showStatus("No projects returned from server.", "error");
146156
}

src/mcp/resources/init_ui.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ export const init_ui = resource(
1313
description: "Visual interface for Firebase Init",
1414
mimeType: RESOURCE_MIME_TYPE,
1515
},
16-
async (uri: string, ctx: McpContext) => {
16+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
17+
async (_uri: string, _ctx: McpContext) => {
1718
try {
1819
// The built HTML will be in lib/mcp/apps/init/mcp-app.html
1920
const htmlPath = path.join(__dirname, "../apps/init/mcp-app.html");

src/mcp/resources/update_environment_ui.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ export const update_environment_ui = resource(
1313
description: "Visual interface for selecting active Firebase project",
1414
mimeType: RESOURCE_MIME_TYPE,
1515
},
16-
async (uri: string, ctx: McpContext) => {
16+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
17+
async (_uri: string, _ctx: McpContext) => {
1718
try {
1819
const htmlPath = path.join(__dirname, "../apps/update_environment/mcp-app.html");
1920
const html = await fs.readFile(htmlPath, "utf-8");

0 commit comments

Comments
 (0)