Merge pull request #95 from Sigmapitech/feat/issue-79/(re)actions

Ciznia · web-flow · commit 2df9ea5fbe84 · 2025-11-05T09:40:47.000+01:00
fix(front): add node deletion
diff --git a/back/app/db/models/oauth.py b/back/app/db/models/oauth.py
@@ -12,7 +12,7 @@ class OAuthToken(Base):
     access_token = Column(String(128), nullable=False)
     refresh_token = Column(String(128), nullable=True)
 
-    service_id = Column(Integer, ForeignKey("service.id"), nullable=False)
+    service = Column(String, nullable=False)
 
     scope = Column(String(512), nullable=True)
     expires_at = Column(DateTime, nullable=True)
diff --git a/back/app/db/models/user.py b/back/app/db/models/user.py
@@ -30,5 +30,5 @@ class User(Base):
         back_populates="user",
         cascade="all, delete-orphan",
         passive_deletes=True,
-        lazy="selectin"
+        lazy="selectin",
     )
diff --git a/back/app/routes/__init__.py b/back/app/routes/__init__.py
@@ -4,6 +4,7 @@
 
 routers = []
 providers = (
+    "about",
     "auth",
     "hello",
     "workflow",
@@ -19,10 +20,12 @@
     try:
         mod = importlib.import_module(f".{mod_name}", __package__)
 
-        assert hasattr(mod, "router"), \
-            f"Module {mod.__name__} is missing 'router' attribute"
-        assert isinstance(mod.router, APIRouter), \
-            f"'router' in module {mod.__name__} is not an APIRouter instance"
+        assert hasattr(
+            mod, "router"
+        ), f"Module {mod.__name__} is missing 'router' attribute"
+        assert isinstance(
+            mod.router, APIRouter
+        ), f"'router' in module {mod.__name__} is not an APIRouter instance"
         print(
             "Registering router from module:"
             f" {mod.__name__} with prefix: {mod.router.prefix}"
diff --git a/back/app/routes/about.py b/back/app/routes/about.py
@@ -0,0 +1,33 @@
+import time
+
+from fastapi import APIRouter, Request
+from fastapi.responses import JSONResponse
+
+from .oauth_base import OAuthProvider
+
+router = APIRouter(prefix="")
+
+
+@router.get("/about.json")
+async def about_json(request: Request):
+    client = request.client
+    client_ip = client.host if client else "unknown"
+    services = OAuthProvider.services.keys()
+    return JSONResponse(
+        {
+            "client": {
+                "host": client_ip,
+            },
+            "server": {
+                "current_time": int(time.time()),
+                "services": [
+                    {
+                        "name": service_name,
+                        "actions": [],
+                        "reactions": [],
+                    }
+                    for service_name in services
+                ],
+            },
+        }
+    )
diff --git a/back/app/routes/auth.py b/back/app/routes/auth.py
@@ -64,7 +64,7 @@ async def login_user(
     },
 )
 async def get_me(
-    current_user = Depends(get_current_user),
+    current_user=Depends(get_current_user),
 ) -> UserSchema:
     connected_services = {token.service: True for token in current_user.tokens}
 
@@ -73,6 +73,7 @@ async def get_me(
         services=connected_services,
     )
 
+
 @router.patch(
     "/credentials",
     response_model=UserBase,
diff --git a/back/app/routes/caldav/google.py b/back/app/routes/caldav/google.py
@@ -47,7 +47,7 @@ class Config(BaseModel):
 provider = OAuthProvider(
     package=__package__,
     config_model=Config,
-    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()
+    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),
 )
 
 
diff --git a/back/app/routes/discord/discord.py b/back/app/routes/discord/discord.py
@@ -33,7 +33,7 @@ class Config(BaseModel):
 provider = OAuthProvider(
     package=__package__,
     config_model=Config,
-    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()
+    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),
 )
 
 
diff --git a/back/app/routes/gmail/gmail.py b/back/app/routes/gmail/gmail.py
@@ -49,7 +49,7 @@ class Config(BaseModel):
 provider = OAuthProvider(
     package=__package__,
     config_model=Config,
-    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()
+    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),
 )
 
 
diff --git a/back/app/routes/oauth_base.py b/back/app/routes/oauth_base.py
@@ -44,12 +44,7 @@ class OAuthProvider:
 
     services = {}
 
-    def __init__(
-        self,
-        icon: str,
-        package: str | None,
-        config_model: Any
-    ):
+    def __init__(self, icon: str, package: str | None, config_model: Any):
         assert package is not None, "Package name must be provided"
 
         *_, service_name = package.split(".")
@@ -172,7 +167,7 @@ async def auth(self, code: str, state: str, db: AsyncSession):
         tokens = resp.json()
 
         token = OAuthToken(
-            user_id=user.id,
+            owner_id=user.id,
             service=self.cfg.service,
             access_token=tokens.get("access_token"),
             refresh_token=tokens.get("refresh_token"),
@@ -294,6 +289,7 @@ async def me(self, user: User, db: AsyncSession):
 
 router = APIRouter(prefix="/services", tags=["services"])
 
+
 @router.get("", response_model=dict[str, str])
 async def get_service_list():
     return OAuthProvider.services
diff --git a/back/app/routes/spotify/spotify.py b/back/app/routes/spotify/spotify.py
@@ -29,7 +29,7 @@ class Config(BaseModel):
 provider = OAuthProvider(
     package=__package__,
     config_model=Config,
-    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()
+    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),
 )
 
 
diff --git a/back/app/routes/youtube/youtube.py b/back/app/routes/youtube/youtube.py
@@ -40,9 +40,10 @@ class Config(BaseModel):
 provider = OAuthProvider(
     package=__package__,
     config_model=Config,
-    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()
+    icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),
 )
 
+
 @router.get("/connect")
 async def youtube_connect(token: str = Query(...), platform: str = Query(...)):
     return await provider.connect(token, platform)
diff --git a/back/app/service.py b/back/app/service.py
@@ -10,24 +10,32 @@ def __init__(self, name: str, description: str):
 
     def action(self, name: str, description: str):
         """Decorator to register an action with metadata."""
+
         def wrapper(func: Callable):
-            self.actions.append({
-                "name": name,
-                "description": description,
-                "function": func,
-            })
+            self.actions.append(
+                {
+                    "name": name,
+                    "description": description,
+                    "function": func,
+                }
+            )
             return func
+
         return wrapper
 
     def reaction(self, name: str, description: str):
         """Decorator to register a reaction with metadata."""
+
         def wrapper(func: Callable):
-            self.reactions.append({
-                "name": name,
-                "description": description,
-                "function": func,
-            })
+            self.reactions.append(
+                {
+                    "name": name,
+                    "description": description,
+                    "function": func,
+                }
+            )
             return func
+
         return wrapper
 
     def to_dict(self) -> dict[str, Any]:
diff --git a/front/src/routes/not-found/index.tsx b/front/src/routes/not-found/index.tsx
@@ -0,0 +1,17 @@
+import { Link } from "react-router";
+import "./style.scss";
+
+export default function NotFoundPage() {
+  return (
+    <div className="notfound-page">
+      <div className="notfound-content">
+        <h1>404</h1>
+        <h2>Page Not Found</h2>
+        <p>Sorry, the page you are looking for does not exist.</p>
+        <Link to="/" className="btn">
+          Go Home
+        </Link>
+      </div>
+    </div>
+  );
+}
diff --git a/front/src/routes/not-found/style.scss b/front/src/routes/not-found/style.scss
@@ -0,0 +1,30 @@
+.notfound-page {
+  display: flex;
+  justify-content: center;
+  align-items: center;
+  width: 100%;
+  height: 100vh;
+  text-align: center;
+  padding: 1em;
+  box-sizing: border-box;
+
+  .notfound-content {
+    h1 {
+      font-size: 6rem;
+      margin: 0;
+      color: #e4bef8;
+    }
+
+    h2 {
+      font-size: 2rem;
+      margin: 10px 0;
+      color: #c6d0f5;
+    }
+
+    p {
+      margin: 15px 0 25px;
+      color: #c6d0f5;
+      font-size: 1rem;
+    }
+  }
+}
diff --git a/front/src/routes/workflow-create/index.tsx b/front/src/routes/workflow-create/index.tsx
diff --git a/front/src/routes/workflow-create/style.scss b/front/src/routes/workflow-create/style.scss
diff --git a/front/src/routes/workflow/index.tsx b/front/src/routes/workflow/index.tsx
diff --git a/front/src/routes/workflow/style.scss b/front/src/routes/workflow/style.scss
diff --git a/front/src/web/entrypoint.tsx b/front/src/web/entrypoint.tsx

Original file line number	Diff line number	Diff line change
`@@ -30,5 +30,5 @@ class User(Base):`
`30`	`30`	`back_populates="user",`
`31`	`31`	`cascade="all, delete-orphan",`
`32`	`32`	`passive_deletes=True,`
`33`		`- lazy="selectin"`
	`33`	`+ lazy="selectin",`
`34`	`34`	`)`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ async def login_user(`
`64`	`64`	`},`
`65`	`65`	`)`
`66`	`66`	`async def get_me(`
`67`		`- current_user = Depends(get_current_user),`
	`67`	`+ current_user=Depends(get_current_user),`
`68`	`68`	`) -> UserSchema:`
`69`	`69`	`connected_services = {token.service: True for token in current_user.tokens}`
`70`	`70`
`@@ -73,6 +73,7 @@ async def get_me(`
`73`	`73`	`services=connected_services,`
`74`	`74`	`)`
`75`	`75`
	`76`	`+`
`76`	`77`	`@router.patch(`
`77`	`78`	`"/credentials",`
`78`	`79`	`response_model=UserBase,`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ class Config(BaseModel):`
`47`	`47`	`provider = OAuthProvider(`
`48`	`48`	`package=__package__,`
`49`	`49`	`config_model=Config,`
`50`		`- icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()`
	`50`	`+ icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),`
`51`	`51`	`)`
`52`	`52`
`53`	`53`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ class Config(BaseModel):`
`33`	`33`	`provider = OAuthProvider(`
`34`	`34`	`package=__package__,`
`35`	`35`	`config_model=Config,`
`36`		`- icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()`
	`36`	`+ icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),`
`37`	`37`	`)`
`38`	`38`
`39`	`39`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ class Config(BaseModel):`
`49`	`49`	`provider = OAuthProvider(`
`50`	`50`	`package=__package__,`
`51`	`51`	`config_model=Config,`
`52`		`- icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()`
	`52`	`+ icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),`
`53`	`53`	`)`
`54`	`54`
`55`	`55`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ class Config(BaseModel):`
`29`	`29`	`provider = OAuthProvider(`
`30`	`30`	`package=__package__,`
`31`	`31`	`config_model=Config,`
`32`		`- icon=(pathlib.Path(__file__).parent / "icon.svg").read_text()`
	`32`	`+ icon=(pathlib.Path(__file__).parent / "icon.svg").read_text(),`
`33`	`33`	`)`
`34`	`34`
`35`	`35`