diff --git a/prophet/app.py b/prophet/app.py
index b7839b8..78c12b1 100644
--- a/prophet/app.py
+++ b/prophet/app.py
@@ -1,68 +1,29 @@
-import hashlib
import json
-import os
-import pickle
-import re
-from dataclasses import dataclass, field
from datetime import datetime
-from pathlib import Path
-from uuid import uuid4
import feedparser
from fastapi import FastAPI
from fastapi.middleware.cors import CORSMiddleware
-from fastapi.responses import HTMLResponse
+from fastapi.staticfiles import StaticFiles
from fastapi_utils.tasks import repeat_every
-from groq import Groq
+
+from prophet import view
+from prophet.domain.improvement import Improvement
+from prophet.domain.improvement_repo import IImprovementRepo
+from prophet.domain.original import Original
+from prophet.infra.improvement_pickle_repo import ImprovementPickleRepo
+from prophet.llm import LLMClient
BEE_FEED = "https://babylonbee.com/feed"
BEE_FEED_TEST = "test/resources/feed_short.atom" # NOTE: Switch out when done testing
-PICKLE_DIR = "/tmp/pollenprophet"
-
REFRESH_PERIOD = 3600 # between fetching articles, in seconds
-
-@dataclass
-class Original: # BadJoke: Sting
- title: str
- summary: str
- link: str
- date: datetime
- image_link: str | None = None
- id: str = field(init=False)
-
- def _extract_img(self, s: str) -> tuple[str, str]: # [img_link, rest of string]
- img: str
- m = re.match(r'", "", s)
- return (img, rest)
-
- def __post_init__(self):
- self.id = hashlib.sha256(self.link.encode()).hexdigest()
-
- extracted = self._extract_img(self.summary)
- if extracted[0]:
- self.image_link = extracted[0]
- self.summary = extracted[1]
-
-
-@dataclass
-class Improvement: # GoodJoke: Queen
- original: Original
- title: str
- summary: str
- id: str = str(uuid4())
+llm: LLMClient = LLMClient()
+repo: IImprovementRepo = ImprovementPickleRepo()
def grab_latest_originals() -> list[Original]:
- # TODO: Implement skipping any we already have
feed: feedparser.FeedParserDict = feedparser.parse(BEE_FEED) # noqa: F841
results: list[Original] = []
for entry in feed.entries:
@@ -76,39 +37,11 @@ def grab_latest_originals() -> list[Original]:
return results
-def save_new_improvements(improvements: list[Improvement]) -> None:
- save_dir = Path(PICKLE_DIR)
- save_dir.mkdir(parents=True, exist_ok=True)
- for imp in improvements:
- fname = save_dir / f"{int(imp.original.date.timestamp())}_{imp.id}"
- try:
- with open(fname, "wb") as f:
- pickle.dump(imp, f)
- print(f"Saved {fname}")
- except Exception as e:
- print(f"Error saving file {fname}: {e}")
-
-
-def load_existing_improvements() -> list[Improvement]:
- improvements: list[Improvement] = []
- for fname in Path(PICKLE_DIR).iterdir():
- if not fname.is_file():
- continue
-
- try:
- with open(fname, "rb") as f:
- obj: Improvement = pickle.load(f)
- improvements.append(obj)
- except FileNotFoundError as e:
- print(f"Error loading file {fname}: {e}")
- return improvements
-
-
def keep_only_new_originals(
additional: list[Original], existing: list[Original] | None = None
):
if not existing:
- existing = [e.original for e in load_existing_improvements()]
+ existing = [e.original for e in repo.get_all()]
existing_hashes = set([e.id for e in existing])
@@ -123,8 +56,8 @@ def keep_only_new_originals(
def improve_originals(originals: list[Original]) -> list[Improvement]:
improvements: list[Improvement] = []
for orig in originals:
- new_title = rewrite_title_with_groq(orig.title)
- new_summary = rewrite_summary_with_groq(orig, new_title)
+ new_title = llm.rewrite_title(orig.title)
+ new_summary = llm.rewrite_summary(orig, new_title)
improvements.append(
Improvement(original=orig, title=new_title, summary=new_summary)
@@ -132,84 +65,32 @@ def improve_originals(originals: list[Original]) -> list[Improvement]:
return improvements
-def rewrite_title_with_groq(original_content: str) -> str:
- client = Groq(api_key=os.getenv("GROQ_API_KEY", "NO_API_KEY_FOUND"))
+def init() -> FastAPI:
+ app = FastAPI()
+ app.mount("/static", StaticFiles(directory="static"), name="static")
- suggestions = client.chat.completions.create(
- messages=[
- {
- "role": "system",
- "content": "You are a comedy writer at a satirical newspaper. Improve on the following satirical headline. Your new headline is funny, can involve current political events and has an edge to it. Print only the suggestions, with one suggestion on each line.",
- },
- {
- "role": "user",
- "content": original_content,
- },
- ],
- model="llama-3.3-70b-versatile",
+ origins = [
+ "http://localhost",
+ "http://localhost:8080",
+ ]
+
+ app.add_middleware(
+ CORSMiddleware,
+ allow_origins=origins,
+ allow_credentials=True,
+ allow_methods=["*"],
+ allow_headers=["*"],
)
- suggestions_str = suggestions.choices[0].message.content
- if not suggestions_str:
- raise ValueError
- print("Suggestions: ", suggestions_str)
- winner = client.chat.completions.create(
- messages=[
- {
- "role": "system",
- "content": "You are an editor at a satirical newspaper. Improve on the following satirical headline. For a given headline, you diligently evaluate: (1) Whether the headline is funny; (2) Whether the headline follows a clear satirical goal; (3) Whether the headline has sufficient substance and bite. Based on the outcomes of your review, you pick your favorite headline from the given suggestions and you make targeted revisions to it. Your output consists solely of the revised headline.",
- },
- {
- "role": "user",
- "content": suggestions_str,
- },
- ],
- model="llama-3.3-70b-versatile",
- )
- print("Winner: ", winner.choices[0].message.content)
- winner_str = winner.choices[0].message.content
- if not winner_str:
- raise ValueError
- return winner_str.strip(" \"'")
+ view.define_routes(app)
+ return app
-def rewrite_summary_with_groq(orig: Original, improved_title: str) -> str:
- client = Groq(api_key=os.getenv("GROQ_API_KEY", "NO_API_KEY_FOUND"))
-
- summary = client.chat.completions.create(
- messages=[
- {
- "role": "user",
- "content": f"Below there is an original title and an original summary. Then follows an improved title. Write an improved summary based on the original summary which fits to the improved title. Only output the improved summary.\n\nTitle:{orig.title}\nSummary:{orig.summary}\n---\nTitle:{improved_title}\nSummary:",
- }
- ],
- model="llama-3.3-70b-versatile",
- )
- summary_str = summary.choices[0].message.content
- if not summary_str:
- raise ValueError
- print("Improved summary", summary_str)
- return summary_str.strip(" \"'")
-
-
-app = FastAPI()
-
-origins = [
- "http://localhost",
- "http://localhost:8080",
-]
-
-app.add_middleware(
- CORSMiddleware,
- allow_origins=origins,
- allow_credentials=True,
- allow_methods=["*"],
- allow_headers=["*"],
-)
+app = init()
@app.get("/improve-title")
def improve_headline(content: str):
- return rewrite_title_with_groq(content)
+ return llm.rewrite_title(content)
@app.get("/improve-summary")
@@ -217,99 +98,26 @@ def improve_summary(original_title: str, new_title: str, original_summary: str):
o = Original(
title=original_title, summary=original_summary, link="", date=datetime.now()
)
- return rewrite_summary_with_groq(o, new_title)
+ return llm.rewrite_summary(o, new_title)
+# TODO: Switch to lifecycle events to avoid deprecated method
@app.on_event("startup")
@repeat_every(seconds=REFRESH_PERIOD)
-def refresh_articles():
- adding = keep_only_new_originals(grab_latest_originals())
- improved = improve_originals(adding)
- save_new_improvements(improved)
- print(f"Updated articles. Added {len(improved)} new ones.")
+async def refresh_articles():
+ _ = await fetch_update()
@app.get("/update")
-async def fetch_update():
- await refresh_articles()
+async def fetch_update(debug_print: bool = True):
+ adding = keep_only_new_originals(grab_latest_originals())
+ improved = improve_originals(adding)
+ repo.add_all(improved)
+ if debug_print:
+ print(f"Updated articles. Added {len(improved)} new ones.")
return json.dumps(improved)
-@app.get("/improvements", response_class=HTMLResponse)
-def list_improvements():
- improved = load_existing_improvements()
- return (
- """ """
- + "\n".join(
- f"""
-